[ https://issues.apache.org/jira/browse/IGNITE-24979?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Andrey Mashenkov updated IGNITE-24979: -------------------------------------- Description: As for now, to convert values into SQL engine internal representation and back, we rely on instance of method to detect value type. In most cases, we already have native type enum and could use it for value conversion. But by no reason, we convert enum to Class object and go non-optimal way with branchy if-tree and `instance of` calls. Let's replace or add optimized version of`TypeUtils.toInternal()` and `TypeUtils.frominternal()` Also, SafeCustomTypeInternalConversion uses HasMap for Enum keys instead of EnumMap. Let's fix this. was: As for now, to convert values into SQL engine internal representation and back, we rely on instance of method to detect value type. In most cases, we already have native type enum and could use it for value conversion. But by no reason, we convert enum to Class object and go non-optimal way with branchy if-tree and `instance of` calls. Let's replace or add optimized version of`TypeUtils.toInternal()` and `TypeUtils.frominternal()` > Sql. Optimize row values conversion > ----------------------------------- > > Key: IGNITE-24979 > URL: https://issues.apache.org/jira/browse/IGNITE-24979 > Project: Ignite > Issue Type: Improvement > Components: sql > Reporter: Andrey Mashenkov > Priority: Major > Labels: ignite-3, performance > Fix For: 3.1 > > > As for now, to convert values into SQL engine internal representation and > back, we rely on instance of method to detect value type. > In most cases, we already have native type enum and could use it for value > conversion. But by no reason, we convert enum to Class object and go > non-optimal way with branchy if-tree and `instance of` calls. > Let's replace or add optimized version of`TypeUtils.toInternal()` and > `TypeUtils.frominternal()` > Also, SafeCustomTypeInternalConversion uses HasMap for Enum keys instead of > EnumMap. > Let's fix this. -- This message was sent by Atlassian Jira (v8.20.10#820010)