You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kylin.apache.org by "Shaofeng SHI (JIRA)" <ji...@apache.org> on 2019/06/30 10:32:00 UTC
[jira] [Created] (KYLIN-4062) Too many "if else" clause in
PushDownRunnerJdbcImpl#toSqlType
Shaofeng SHI created KYLIN-4062:
-----------------------------------
Summary: Too many "if else" clause in PushDownRunnerJdbcImpl#toSqlType
Key: KYLIN-4062
URL: https://issues.apache.org/jira/browse/KYLIN-4062
Project: Kylin
Issue Type: Improvement
Components: Query Engine
Reporter: Shaofeng SHI
In this method, it has 30 "if else" which is low efficient; Should use a static Hashmap, then only need 1 check.
{code:java}
if ("string".equalsIgnoreCase(type)) {
return Types.VARCHAR;
} else if ("varchar".equalsIgnoreCase(type)) {
return Types.VARCHAR;
} else if ("char".equalsIgnoreCase(type)) {
return Types.CHAR;
} else if
...{code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)