[
https://issues.apache.org/jira/browse/KYLIN-3025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16524722#comment-16524722
]
ASF GitHub Bot commented on KYLIN-3025:
---------------------------------------
shaofengshi closed pull request #158: KYLIN-3025 fix error: {fn CONVERT} for
bigint type in tableau 10.4
URL: https://github.com/apache/kylin/pull/158
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git
a/query/src/main/java/org/apache/kylin/query/util/DefaultQueryTransformer.java
b/query/src/main/java/org/apache/kylin/query/util/DefaultQueryTransformer.java
index f8ab8f24d6..f32e562cb6 100644
---
a/query/src/main/java/org/apache/kylin/query/util/DefaultQueryTransformer.java
+++
b/query/src/main/java/org/apache/kylin/query/util/DefaultQueryTransformer.java
@@ -49,7 +49,7 @@
+ S0 + "([^\\s,]+)" + S0 + "AS" + SM + "DOUBLE" + S0 + "\\)" + S0
+ "\\)", Pattern.CASE_INSENSITIVE);
private static final Pattern PIN_SUM_OF_FN_CONVERT = Pattern
.compile(S0 + "SUM" + S0 + "\\(" + S0 + "\\{\\s*fn" + SM +
"convert" + S0 + "\\(" + S0 + "([^\\s,]+)" + S0
- + "," + S0 + "SQL_DOUBLE" + S0 + "\\)" + S0 + "\\}" + S0 +
"\\)", Pattern.CASE_INSENSITIVE);
+ + "," + S0 + "(SQL_DOUBLE|SQL_BIGINT)" + S0 + "\\)" + S0 +
"\\}" + S0 + "\\)", Pattern.CASE_INSENSITIVE);
@Override
public String transform(String sql, String project, String defaultSchema) {
diff --git
a/query/src/test/java/org/apache/kylin/query/util/DefaultQueryTransformerTest.java
b/query/src/test/java/org/apache/kylin/query/util/DefaultQueryTransformerTest.java
index b6265b3795..a38b887e44 100644
---
a/query/src/test/java/org/apache/kylin/query/util/DefaultQueryTransformerTest.java
+++
b/query/src/test/java/org/apache/kylin/query/util/DefaultQueryTransformerTest.java
@@ -27,39 +27,43 @@
@Test
public void SumOfFnConvertTransform() throws Exception {
+ SumOfFnConvertTransform("SQL_DOUBLE");
+ SumOfFnConvertTransform("SQL_BIGINT");
+ }
+
+ private void SumOfFnConvertTransform(String dataType) throws Exception {
DefaultQueryTransformer transformer = new DefaultQueryTransformer();
- String fnConvertSumSql = "select sum({fn convert(\"LSTG_SITE_ID\",
SQL_DOUBLE)}) from KYLIN_SALES group by LSTG_SITE_ID";
+ String fnConvertSumSql = "select sum({fn convert(\"LSTG_SITE_ID\", " +
dataType + ")}) from KYLIN_SALES group by LSTG_SITE_ID";
String correctSql = transformer.transform(fnConvertSumSql, "", "");
assertTrue("select sum(\"LSTG_SITE_ID\") from KYLIN_SALES group by
LSTG_SITE_ID".equalsIgnoreCase(correctSql));
//test SQL contains blank
//Case one blank interval
- fnConvertSumSql = "select sum ( { fn convert( \"LSTG_SITE_ID\" ,
SQL_DOUBLE) } ) from KYLIN_SALES group by LSTG_SITE_ID";
+ fnConvertSumSql = "select sum ( { fn convert( \"LSTG_SITE_ID\" , " +
dataType + ") } ) from KYLIN_SALES group by LSTG_SITE_ID";
correctSql = transformer.transform(fnConvertSumSql, "", "");
assertTrue("select sum(\"LSTG_SITE_ID\") from KYLIN_SALES group by
LSTG_SITE_ID".equalsIgnoreCase(correctSql));
//Case multi blank interval
- fnConvertSumSql = "select SUM ( { fn convert( \"LSTG_SITE_ID\" ,
SQL_DOUBLE ) } ) from KYLIN_SALES group by LSTG_SITE_ID";
+ fnConvertSumSql = "select SUM ( { fn convert( \"LSTG_SITE_ID\" ,
" + dataType + " ) } ) from KYLIN_SALES group by LSTG_SITE_ID";
correctSql = transformer.transform(fnConvertSumSql, "", "");
assertTrue("select sum(\"LSTG_SITE_ID\") from KYLIN_SALES group by
LSTG_SITE_ID".equalsIgnoreCase(correctSql));
//Case one or multi blank interval
- fnConvertSumSql = "select SUM( { fn convert( \"LSTG_SITE_ID\" ,
SQL_DOUBLE ) } ) from KYLIN_SALES group by LSTG_SITE_ID";
+ fnConvertSumSql = "select SUM( { fn convert( \"LSTG_SITE_ID\" , " +
dataType + " ) } ) from KYLIN_SALES group by LSTG_SITE_ID";
correctSql = transformer.transform(fnConvertSumSql, "", "");
assertTrue("select sum(\"LSTG_SITE_ID\") from KYLIN_SALES group by
LSTG_SITE_ID".equalsIgnoreCase(correctSql));
//test exception case of "... fnconvert ..."
- fnConvertSumSql = "select SUM ({fnconvert(\"LSTG_SITE_ID\",
SQL_DOUBLE)}) from KYLIN_SALES group by LSTG_SITE_ID";
+ fnConvertSumSql = "select SUM ({fnconvert(\"LSTG_SITE_ID\", " +
dataType + ")}) from KYLIN_SALES group by LSTG_SITE_ID";
correctSql = transformer.transform(fnConvertSumSql, "", "");
assertFalse("select sum(\"LSTG_SITE_ID\") from KYLIN_SALES group by
LSTG_SITE_ID".equalsIgnoreCase(correctSql));
//test SQL contains multi sum
- fnConvertSumSql = "select SUM({fn convert(\"LSTG_SITE_ID\",
SQL_DOUBLE)}), SUM({fn convert(\"price\", SQL_DOUBLE)}) from KYLIN_SALES group
by LSTG_SITE_ID";
+ fnConvertSumSql = "select SUM({fn convert(\"LSTG_SITE_ID\", " +
dataType + ")}), SUM({fn convert(\"price\", " + dataType + ")}) from
KYLIN_SALES group by LSTG_SITE_ID";
correctSql = transformer.transform(fnConvertSumSql, "", "");
assertTrue("select sum(\"LSTG_SITE_ID\"), sum(\"price\") from
KYLIN_SALES group by LSTG_SITE_ID"
.equalsIgnoreCase(correctSql));
-
}
@Test
@@ -92,7 +96,7 @@ public void SumOfCastTransform() throws Exception {
assertTrue("select sum(LSTG_SITE_ID), sum(price) from KYLIN_SALES
group by LSTG_SITE_ID"
.equalsIgnoreCase(correctSql));
}
-
+
@Test
public void functionEscapeTransform() throws Exception {
DefaultQueryTransformer transformer = new DefaultQueryTransformer();
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> kylin odbc error : {fn CONVERT} for bigint type in tableau 10.4
> ---------------------------------------------------------------
>
> Key: KYLIN-3025
> URL: https://issues.apache.org/jira/browse/KYLIN-3025
> Project: Kylin
> Issue Type: Bug
> Components: Driver - ODBC
> Affects Versions: v2.1.0
> Environment: Windows 10;kylin v2.2.0;tableau desktop 10.4;kylin odbc
> v2.1.0
> Reporter: Hokyung Song
> Assignee: Chao Long
> Priority: Major
> Fix For: v2.5.0
>
>
> In Tableau Desktop version 9.3, also it works very well.
> but..when I use measure field (bigint type of data) in Tableau Desktop
> version 10.4, it occured query error. (decimal type field works well.)
> I guess.. "{fn CONVERT}" function couldn't parsed. please support tableau
> 10.4 :)
> Detailed query example from tableau:
> {code}
> SELECT "MOMENT"."P_DT" AS "P_DT",
> SUM({fn CONVERT("MOMENT"."ACTIONCOUNT", SQL_BIGINT)}) AS
> "sum_ACTIONCOUNT_ok"
> FROM "TEST"."MOMENT" "MOMENT"
> GROUP BY "MOMENT"."P_DT"
> {code}
> ----
> environment
> ----
> * kylin server : v2.2.0
> * kylin odbc (for windows client) : v2.1.0
> * tableau desktop : 10.4
> * client os : windows 10 (64bit)
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
