This is an automated email from the ASF dual-hosted git repository.
xxyu pushed a commit to branch kylin5-beta-temp
in repository https://gitbox.apache.org/repos/asf/kylin.git
The following commit(s) were added to refs/heads/kylin5-beta-temp by this push:
new 8983bf45b4 Fix NPE for BasicService#getCurrentUserGroups
8983bf45b4 is described below
commit 8983bf45b4fed37e6aa0a5321389ffbc14edf800
Author: XiaoxiangYu <x...@apache.org>
AuthorDate: Mon Aug 14 16:28:50 2023 +0800
Fix NPE for BasicService#getCurrentUserGroups
---
.../src/main/java/org/apache/kylin/common/KylinConfigBase.java | 1 -
.../main/java/org/apache/kylin/metadata/project/ProjectInstance.java | 1 -
.../src/main/java/org/apache/kylin/metadata/query/QueryHistory.java | 2 +-
.../src/main/java/org/apache/kylin/metadata/query/QueryMetrics.java | 1 -
.../kylin/source/adhocquery/DoubleQuotePushDownConverterTest.java | 2 +-
.../test/java/org/apache/kylin/newten/NBadQueryAndPushDownTest.java | 1 -
.../test/java/org/apache/kylin/query/engine/SumExprPlannerTest.java | 1 -
src/kylin-it/src/test/resources/query/sql/query135.sql | 1 -
src/kylin-it/src/test/resources/query/sql/query136.sql | 1 -
src/kylin-it/src/test/resources/query/sql/query139.sql | 1 -
src/kylin-it/src/test/resources/query/sql_casewhen/query11.sql | 1 -
src/kylin-it/src/test/resources/query/sql_casewhen/query12.sql | 1 -
.../query/sql_computedcolumn/sql_computedcolumn_StringUDF/query06.sql | 1 -
src/kylin-it/src/test/resources/query/sql_function/query36.sql | 1 -
src/kylin-it/src/test/resources/query/sql_function/query37.sql | 1 -
.../resources/query/sql_join/sql_is_not_distinct_from/query_01.sql | 2 +-
.../resources/query/sql_join/sql_is_not_distinct_from/query_02.sql | 2 +-
.../resources/query/sql_join/sql_is_not_distinct_from/query_03.sql | 2 +-
.../resources/query/sql_join/sql_is_not_distinct_from/query_04.sql | 2 +-
.../resources/query/sql_join/sql_is_not_distinct_from/query_05.sql | 2 +-
src/kylin-it/src/test/resources/query/sql_sum_expr/query00.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query25.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query26.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query27.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query28.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query29.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query30.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query31.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query32.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query33.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query34.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query35.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query36.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query37.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query38.sql.ignore | 2 --
src/kylin-it/src/test/resources/query/sql_sum_expr/query39.sql | 1 -
src/kylin-it/src/test/resources/query/sql_sum_expr/query40.sql | 2 +-
.../main/java/org/apache/kylin/rest/service/ModelSemanticHelper.java | 4 +++-
.../src/main/java/org/apache/kylin/rest/service/ModelService.java | 2 +-
.../src/test/java/org/apache/kylin/rest/service/ModelServiceTest.java | 1 -
.../java/org/apache/kylin/query/util/RestoreFromComputedColumn.java | 1 -
.../src/main/java/org/apache/kylin/rest/service/QueryService.java | 4 ++--
.../src/main/java/org/apache/kylin/rest/util/SparderUIUtil.java | 1 -
.../java/org/apache/kylin/rest/metrics/QueryMetricsContextTest.java | 1 -
.../src/test/java/org/apache/kylin/rest/service/QueryServiceTest.java | 2 --
.../java/org/apache/kylin/query/engine/AsyncQueryApplication.java | 2 +-
.../java/io/kyligence/kap/secondstorage/abnormal/QueryRouteTest.java | 1 -
.../io/kyligence/kap/secondstorage/abnormal/SecondaryCatalogTest.java | 1 -
.../src/main/java/org/apache/kylin/job/SecondStorageJobParamUtil.java | 1 -
.../org/apache/kylin/engine/spark/merger/SparkJobMetadataMerger.java | 2 +-
.../org/apache/kylin/engine/spark/builder/DFDictionaryBuilder.scala | 1 -
.../kylin/engine/spark/job/stage/build/FlatTableAndDictBase.scala | 2 +-
.../java/org/apache/kylin/engine/spark/source/NSparkSourceTest.java | 1 -
.../sparder/src/main/scala/org/apache/spark/sql/KylinSession.scala | 1 -
.../src/test/scala/org/apache/spark/sql/udf/TimestampDiffTest.scala | 1 -
.../main/java/org/apache/kylin/tool/kerberos/KerberosLoginTask.java | 2 +-
56 files changed, 18 insertions(+), 59 deletions(-)
diff --git
a/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java
b/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java
index 23d53d3367..785fa9a048 100644
--- a/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java
+++ b/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java
@@ -1844,7 +1844,6 @@ public abstract class KylinConfigBase implements
Serializable {
}
// If return empty result for select star query
- // https://olapio.atlassian.net/browse/KE-23663
public boolean getEmptyResultForSelectStar() {
return
Boolean.parseBoolean(getOptional("kylin.query.return-empty-result-on-select-star",
FALSE));
}
diff --git
a/src/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectInstance.java
b/src/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectInstance.java
index 07183ebce7..9b0c36292a 100644
---
a/src/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectInstance.java
+++
b/src/core-metadata/src/main/java/org/apache/kylin/metadata/project/ProjectInstance.java
@@ -137,7 +137,6 @@ public class ProjectInstance extends RootPersistentEntity
implements ISourceAwar
}
public void initConfig(KylinConfig config) {
- // https://olapio.atlassian.net/browse/KE-11535
// to compatible with existing model
// if expose-computed-column is empty, set it with the
maintainModelType
if (!overrideKylinProps.containsKey(EXPOSE_COMPUTED_COLUMN_CONF)) {
diff --git
a/src/core-metadata/src/main/java/org/apache/kylin/metadata/query/QueryHistory.java
b/src/core-metadata/src/main/java/org/apache/kylin/metadata/query/QueryHistory.java
index fd78323f0d..59b2b3f0c5 100644
---
a/src/core-metadata/src/main/java/org/apache/kylin/metadata/query/QueryHistory.java
+++
b/src/core-metadata/src/main/java/org/apache/kylin/metadata/query/QueryHistory.java
@@ -107,7 +107,7 @@ public class QueryHistory {
// this field is composed of modelId, layout id and index type
// it's written as modelId#layoutId#indexType
- // This way to serialized query realizations had been deprecated. See
KE-20697
+ // This way to serialized query realizations had been deprecated.
private String queryRealizations;
@JsonProperty(QUERY_SERVER)
diff --git
a/src/core-metadata/src/main/java/org/apache/kylin/metadata/query/QueryMetrics.java
b/src/core-metadata/src/main/java/org/apache/kylin/metadata/query/QueryMetrics.java
index 031bfc2f06..4e9eec2bb9 100644
---
a/src/core-metadata/src/main/java/org/apache/kylin/metadata/query/QueryMetrics.java
+++
b/src/core-metadata/src/main/java/org/apache/kylin/metadata/query/QueryMetrics.java
@@ -54,7 +54,6 @@ public class QueryMetrics extends SchedulerEventNotifier {
protected String projectName;
protected String sql;
- // KE-36662 Using sql_pattern as normalized_sql storage
protected String sqlPattern;
protected String submitter;
diff --git
a/src/core-metadata/src/test/java/org/apache/kylin/source/adhocquery/DoubleQuotePushDownConverterTest.java
b/src/core-metadata/src/test/java/org/apache/kylin/source/adhocquery/DoubleQuotePushDownConverterTest.java
index 190ad3bce3..e49b67b6e5 100644
---
a/src/core-metadata/src/test/java/org/apache/kylin/source/adhocquery/DoubleQuotePushDownConverterTest.java
+++
b/src/core-metadata/src/test/java/org/apache/kylin/source/adhocquery/DoubleQuotePushDownConverterTest.java
@@ -84,7 +84,7 @@ public class DoubleQuotePushDownConverterTest extends
NLocalFileMetadataTestCase
Arrays.asList("select a + b * c", "select \"A\" + \"B\" *
\"C\""),
Arrays.asList("select 1 + b * c", "select 1 + \"B\" * \"C\""),
- //for filter the function without parentheses,ref KE-13407
+ //for filter the function without parentheses
Arrays.asList(
"select CURRENT_CATALOG, CURRENT_DATE,
CURRENT_PATH,CURRENT_ROLE, CURRENT_SCHEMA, CURRENT_TIME, CURRENT_TIMESTAMP,
CURRENT_USER,LOCALTIME, LOCALTIMESTAMP, SESSION_USER, SYSTEM_USER, USER",
"select CURRENT_CATALOG, CURRENT_DATE,
CURRENT_PATH,CURRENT_ROLE, CURRENT_SCHEMA, CURRENT_TIME, CURRENT_TIMESTAMP,
CURRENT_USER,LOCALTIME, LOCALTIMESTAMP, SESSION_USER, SYSTEM_USER, USER"),
diff --git
a/src/kylin-it/src/test/java/org/apache/kylin/newten/NBadQueryAndPushDownTest.java
b/src/kylin-it/src/test/java/org/apache/kylin/newten/NBadQueryAndPushDownTest.java
index 58561b6f82..ce0df3858c 100644
---
a/src/kylin-it/src/test/java/org/apache/kylin/newten/NBadQueryAndPushDownTest.java
+++
b/src/kylin-it/src/test/java/org/apache/kylin/newten/NBadQueryAndPushDownTest.java
@@ -203,7 +203,6 @@ public class NBadQueryAndPushDownTest extends
NLocalWithSparkSessionTest {
@Test
public void testPushDownForced() throws Exception {
- //test for KE-14218
KylinConfig.getInstanceFromEnv().setProperty(PUSHDOWN_RUNNER_KEY,
"org.apache.kylin.query.pushdown.PushDownRunnerSparkImpl");
diff --git
a/src/kylin-it/src/test/java/org/apache/kylin/query/engine/SumExprPlannerTest.java
b/src/kylin-it/src/test/java/org/apache/kylin/query/engine/SumExprPlannerTest.java
index 34ebed5f81..7e00b6cfb2 100644
---
a/src/kylin-it/src/test/java/org/apache/kylin/query/engine/SumExprPlannerTest.java
+++
b/src/kylin-it/src/test/java/org/apache/kylin/query/engine/SumExprPlannerTest.java
@@ -113,7 +113,6 @@ public class SumExprPlannerTest extends CalciteRuleTestBase
{
}
/**
- * see https://olapio.atlassian.net/browse/KE-14512
*/
@Test
public void testWithAVG() {
diff --git a/src/kylin-it/src/test/resources/query/sql/query135.sql
b/src/kylin-it/src/test/resources/query/sql/query135.sql
index 0ccd63106a..19dfa30269 100644
--- a/src/kylin-it/src/test/resources/query/sql/query135.sql
+++ b/src/kylin-it/src/test/resources/query/sql/query135.sql
@@ -17,7 +17,6 @@
--
--- KE-14621 division between integers
SELECT {fn LTRIM( {fn CONVERT( {fn CONVERT( {fn TRUNCATE( ( 6 + {fn
DAYOFYEAR(CAL_DT)} + ( {fn MOD( ( 7 + {fn DAYOFWEEK( {fn TIMESTAMPADD(
SQL_TSI_DAY, {fn CONVERT( {fn TRUNCATE( (-1 * ({fn DAYOFYEAR(CAL_DT)} - 1)),
0)}, SQL_BIGINT)}, {fn CONVERT( CAL_DT, SQL_DATE)} )} )} - 2), 7)} ) ) / 7,
0)}, SQL_BIGINT)}, SQL_VARCHAR)})} AS "TEMP_Test__499182808__0_"
FROM TEST_KYLIN_FACT
GROUP BY {fn LTRIM({fn CONVERT({fn CONVERT({fn TRUNCATE((6 + {fn
DAYOFYEAR(CAL_DT)} + ({fn MOD((7 + {fn DAYOFWEEK({fn
TIMESTAMPADD(SQL_TSI_DAY,{fn CONVERT({fn TRUNCATE((-1 * ({fn DAYOFYEAR(CAL_DT)}
- 1)),0)}, SQL_BIGINT)},{fn CONVERT(CAL_DT, SQL_DATE)})})} - 2), 7)})) / 7,0)},
SQL_BIGINT)}, SQL_VARCHAR)})}
diff --git a/src/kylin-it/src/test/resources/query/sql/query136.sql
b/src/kylin-it/src/test/resources/query/sql/query136.sql
index 5b00b0fff9..dfaccbbc85 100644
--- a/src/kylin-it/src/test/resources/query/sql/query136.sql
+++ b/src/kylin-it/src/test/resources/query/sql/query136.sql
@@ -17,7 +17,6 @@
--
--- KE-14621 division between integers
SELECT 5/4 as A,
5.0/4 AS B,
5/4.0 AS C,
diff --git a/src/kylin-it/src/test/resources/query/sql/query139.sql
b/src/kylin-it/src/test/resources/query/sql/query139.sql
index ce32702c43..ca9138aea2 100644
--- a/src/kylin-it/src/test/resources/query/sql/query139.sql
+++ b/src/kylin-it/src/test/resources/query/sql/query139.sql
@@ -16,7 +16,6 @@
-- limitations under the License.
--
--- KE-19732
select floor(cal_dt to day), ceil(cal_dt to day),
floor(cal_dt to week), ceil(cal_dt to week),
diff --git a/src/kylin-it/src/test/resources/query/sql_casewhen/query11.sql
b/src/kylin-it/src/test/resources/query/sql_casewhen/query11.sql
index 84b6a62a0c..d6b5e4221e 100644
--- a/src/kylin-it/src/test/resources/query/sql_casewhen/query11.sql
+++ b/src/kylin-it/src/test/resources/query/sql_casewhen/query11.sql
@@ -13,7 +13,6 @@
-- See the License for the specific language governing permissions and
-- limitations under the License.
--
--- KE-13678
select count(*)
from TEST_KYLIN_FACT
diff --git a/src/kylin-it/src/test/resources/query/sql_casewhen/query12.sql
b/src/kylin-it/src/test/resources/query/sql_casewhen/query12.sql
index 3a499178a9..dd0133018f 100644
--- a/src/kylin-it/src/test/resources/query/sql_casewhen/query12.sql
+++ b/src/kylin-it/src/test/resources/query/sql_casewhen/query12.sql
@@ -13,7 +13,6 @@
-- See the License for the specific language governing permissions and
-- limitations under the License.
--
--- KE-13678
select count(*)
from TEST_KYLIN_FACT
diff --git
a/src/kylin-it/src/test/resources/query/sql_computedcolumn/sql_computedcolumn_StringUDF/query06.sql
b/src/kylin-it/src/test/resources/query/sql_computedcolumn/sql_computedcolumn_StringUDF/query06.sql
index 8f54b61e12..a064dad689 100644
---
a/src/kylin-it/src/test/resources/query/sql_computedcolumn/sql_computedcolumn_StringUDF/query06.sql
+++
b/src/kylin-it/src/test/resources/query/sql_computedcolumn/sql_computedcolumn_StringUDF/query06.sql
@@ -17,7 +17,6 @@
--
--- KE-27687
select
count(repeat(NAME1,2)),
count(repeat(NAME2,3)),
diff --git a/src/kylin-it/src/test/resources/query/sql_function/query36.sql
b/src/kylin-it/src/test/resources/query/sql_function/query36.sql
index 6278cb2eae..a88ffc880a 100644
--- a/src/kylin-it/src/test/resources/query/sql_function/query36.sql
+++ b/src/kylin-it/src/test/resources/query/sql_function/query36.sql
@@ -17,7 +17,6 @@
--
--- KE-19050
select
substring(ACCOUNT_COUNTRY from cast(2.0 as integer) for 2),
substring('abcdefg' from cast(2.0 as integer) for 2)
diff --git a/src/kylin-it/src/test/resources/query/sql_function/query37.sql
b/src/kylin-it/src/test/resources/query/sql_function/query37.sql
index 094582680f..540923ae3f 100644
--- a/src/kylin-it/src/test/resources/query/sql_function/query37.sql
+++ b/src/kylin-it/src/test/resources/query/sql_function/query37.sql
@@ -17,7 +17,6 @@
--
--- KE-27687
select
repeat(NAME1,2),
repeat(NAME2,3),
diff --git
a/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_01.sql
b/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_01.sql
index 04f46180d5..df129b7990 100644
---
a/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_01.sql
+++
b/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_01.sql
@@ -16,7 +16,7 @@
-- limitations under the License.
--
--- KE-19841 optimize non-equal join with is not distinct from condition
+-- optimize non-equal join with is not distinct from condition
SELECT SUM(PRICE),
CAL_DT
FROM TEST_KYLIN_FACT
diff --git
a/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_02.sql
b/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_02.sql
index 0c3a63c05f..fd505d4552 100644
---
a/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_02.sql
+++
b/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_02.sql
@@ -16,7 +16,7 @@
-- limitations under the License.
--
--- KE-19841 optimize non-equal join with is not distinct from condition
+-- optimize non-equal join with is not distinct from condition
SELECT SUM(PRICE),
CAL_DT
FROM TEST_KYLIN_FACT
diff --git
a/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_03.sql
b/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_03.sql
index 0b2450bd80..94d858c85f 100644
---
a/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_03.sql
+++
b/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_03.sql
@@ -16,7 +16,7 @@
-- limitations under the License.
--
--- KE-19841 optimize non-equal join with is not distinct from condition
+-- optimize non-equal join with is not distinct from condition
SELECT SUM(PRICE),
CAL_DT
FROM TEST_KYLIN_FACT
diff --git
a/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_04.sql
b/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_04.sql
index d6fb86ca5d..758df68746 100644
---
a/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_04.sql
+++
b/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_04.sql
@@ -16,7 +16,7 @@
-- limitations under the License.
--
--- KE-19841 optimize non-equal join with is not distinct from condition
+-- optimize non-equal join with is not distinct from condition
SELECT SUM(PRICE),
CAL_DT
FROM TEST_KYLIN_FACT
diff --git
a/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_05.sql
b/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_05.sql
index d6fb86ca5d..758df68746 100644
---
a/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_05.sql
+++
b/src/kylin-it/src/test/resources/query/sql_join/sql_is_not_distinct_from/query_05.sql
@@ -16,7 +16,7 @@
-- limitations under the License.
--
--- KE-19841 optimize non-equal join with is not distinct from condition
+-- optimize non-equal join with is not distinct from condition
SELECT SUM(PRICE),
CAL_DT
FROM TEST_KYLIN_FACT
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query00.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query00.sql
index f10628c80c..05934422b1 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query00.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query00.sql
@@ -19,7 +19,6 @@
-- ISSUE #10686
-- ISSUE #10852
--- ISSUE https://olapio.atlassian.net/browse/KE-8756
SELECT COUNT(*) AS COL1,
SUM(CASE WHEN "TEST_CAL_DT"."CAL_DT" <= CAST('2012-12-31' AS DATE) THEN
PRICE*4
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query25.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query25.sql
index ad0f30fbab..994f3e57b9 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query25.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query25.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-9299
-- Case 1
SELECT
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query26.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query26.sql
index 7451f2995c..2710c709c2 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query26.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query26.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-9299
-- Case 2.1: "PRICE/100" and {fn convert(NULL,DECIMAL(30,15))} types aligned,
hit cube
SELECT
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query27.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query27.sql
index 4883db53f4..cb98846bad 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query27.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query27.sql
@@ -17,7 +17,6 @@
--
--- TODO ISSUE https://olapio.atlassian.net/browse/KE-9299
-- Case 2.2: "PRICE/100" and {fn convert(NULL,DECIMAL(30,15))} types not
aligned, hit index
SELECT
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query28.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query28.sql
index 92add9de27..099dbb0b2d 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query28.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query28.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-9299
-- Case 3
SELECT SUM(CASE WHEN "TEST_CAL_DT"."CAL_DT" <= CAST('2012-12-31' AS DATE) THEN
PRICE/100 WHEN "TEST_CAL_DT"."CAL_DT" > CAST('2012-12-31' AS DATE) THEN 1 ELSE
NULL END) AS COL
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query29.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query29.sql
index a2283ad5e3..644a6a2b03 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query29.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query29.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-8640
-- Case 1
select
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query30.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query30.sql
index 435f7a3334..141d476344 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query30.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query30.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-8640
-- Case 2
select sum(case when LSTG_FORMAT_NAME='ABIN' then price end) AS COL1,
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query31.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query31.sql
index 5b149a38bd..f978b741e5 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query31.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query31.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-8892
SELECT count(SLR_SEGMENT_CD) AS COL1,
SUM(CASE WHEN TEST_KYLIN_FACT.CAL_DT BETWEEN '2011-10-01' AND '2012-10-31'
THEN PRICE ELSE 0 END) as COL2
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query32.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query32.sql
index 611afedd2d..bbb4f8a575 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query32.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query32.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-8378
-- Case 1: Kylin goes well but H2 does not support such query: select sum(case
when 1=2 then 1 end) AS COL
-- modify it as follow:
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query33.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query33.sql
index f232183d46..4bd96f57d5 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query33.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query33.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-8378
-- Case 2
select sum(case when nullif(5,5)=cast(null as int) then PRICE end) AS COL
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query34.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query34.sql
index 80d06e378f..aa2973ff93 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query34.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query34.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-8289
-- Use PRICE as dimension, will hit index
select
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query35.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query35.sql
index 54c6c9b4cf..69a4fbd8b8 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query35.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query35.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-8789
select count(*) AS COL1,
sum(case when CAL_DT = '2012-01-01' then PRICE else 1 end) AS COL2
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query36.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query36.sql
index 9690e246de..c4ae92de28 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query36.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query36.sql
@@ -17,7 +17,6 @@
--
--- ISSUE https://olapio.atlassian.net/browse/KE-9491
select sum(case when CAL_DT>=date '2012-01-01' then price*1 end) as aaa
from TEST_KYLIN_FACT
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query37.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query37.sql
index 650722f49f..1973c217b7 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query37.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query37.sql
@@ -17,7 +17,6 @@
--
--- https://olapio.atlassian.net/browse/KE-12536
SELECT "TEST_CAL_DT"."CAL_DT" AS COL1,
AVG(CASE WHEN LSTG_FORMAT_NAME='ABIN' then 0 else 1 end) AS COL92,
diff --git
a/src/kylin-it/src/test/resources/query/sql_sum_expr/query38.sql.ignore
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query38.sql.ignore
index 6c2026c3c5..f3678c5d43 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query38.sql.ignore
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query38.sql.ignore
@@ -15,8 +15,6 @@
-- See the License for the specific language governing permissions and
-- limitations under the License.
--- https://olapio.atlassian.net/browse/KE-12527
-
SELECT 1.1000000000000001 AS "X__alias__0",
SUM(CAST(1 AS DOUBLE)) AS "sum_Number_of_Records_ok"
FROM "TDVT"."CALCS" "Calcs"
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query39.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query39.sql
index 4af1985173..5ad1683d7f 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query39.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query39.sql
@@ -13,7 +13,6 @@
-- See the License for the specific language governing permissions and
-- limitations under the License.
--
--- KE-14446 sum case when with count distinct
SELECT
COUNT(1),
diff --git a/src/kylin-it/src/test/resources/query/sql_sum_expr/query40.sql
b/src/kylin-it/src/test/resources/query/sql_sum_expr/query40.sql
index 8727151611..c5433589cc 100644
--- a/src/kylin-it/src/test/resources/query/sql_sum_expr/query40.sql
+++ b/src/kylin-it/src/test/resources/query/sql_sum_expr/query40.sql
@@ -13,7 +13,7 @@
-- See the License for the specific language governing permissions and
-- limitations under the License.
--
--- KE-14446 sum case when with percentile
+-- sum case when with percentile
SELECT
ROUND(PERCENTILE(PRICE, 0.5), -1),
diff --git
a/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelSemanticHelper.java
b/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelSemanticHelper.java
index 0badc8532a..e47bfe2eb6 100644
---
a/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelSemanticHelper.java
+++
b/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelSemanticHelper.java
@@ -258,7 +258,9 @@ public class ModelSemanticHelper extends BasicService {
String project = dataModel.getProject();
val projectKylinConfig = NProjectManager.getProjectConfig(project);
boolean isScd2Enabled =
projectKylinConfig.isQueryNonEquiJoinModelEnabled();
-
QueryContext.current().setAclInfo(AclPermissionUtil.createAclInfo(project,
getCurrentUserGroups()));
+ if (!projectKylinConfig.isUTEnv()) {
+
QueryContext.current().setAclInfo(AclPermissionUtil.createAclInfo(project,
getCurrentUserGroups()));
+ }
QueryExec queryExec = new QueryExec(project, projectKylinConfig,
false);
for (int i = 0; i < requestJoinTableDescs.size(); i++) {
final JoinDesc joinWithoutNonEquivInfo =
dataModel.getJoinTables().get(i).getJoin();
diff --git
a/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelService.java
b/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelService.java
index 67ac3e2094..c0fbe9c02f 100644
---
a/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelService.java
+++
b/src/modeling-service/src/main/java/org/apache/kylin/rest/service/ModelService.java
@@ -3907,7 +3907,7 @@ public class ModelService extends AbstractModelService
implements TableModelSupp
/**
* Validate computed column type and throw errors to report wrongly typed
computed columns.
- * Models migrated from 3x may have wrongly typed computed columns. see
KE-11862
+ * Models migrated from 3x may have wrongly typed computed columns.
*
* @param modelId
* @param project
diff --git
a/src/modeling-service/src/test/java/org/apache/kylin/rest/service/ModelServiceTest.java
b/src/modeling-service/src/test/java/org/apache/kylin/rest/service/ModelServiceTest.java
index d8a5616a07..0f6eaba520 100644
---
a/src/modeling-service/src/test/java/org/apache/kylin/rest/service/ModelServiceTest.java
+++
b/src/modeling-service/src/test/java/org/apache/kylin/rest/service/ModelServiceTest.java
@@ -677,7 +677,6 @@ public class ModelServiceTest extends SourceTestCase {
Assert.assertEquals(3, segments.size());
Assert.assertEquals("MERGING",
segments.get(2).getStatusToDisplay().toString());
- // KE-25547, complete segment response
val seg2Resp = segments.stream().filter(s ->
s.getId().equals(seg2.getId())).findFirst().get();
Assert.assertNotNull(seg2Resp);
Assert.assertEquals(seg2.isSnapshotReady(),
seg2Resp.isSnapshotReady());
diff --git
a/src/query-common/src/main/java/org/apache/kylin/query/util/RestoreFromComputedColumn.java
b/src/query-common/src/main/java/org/apache/kylin/query/util/RestoreFromComputedColumn.java
index c388f59ace..148f40fb6c 100644
---
a/src/query-common/src/main/java/org/apache/kylin/query/util/RestoreFromComputedColumn.java
+++
b/src/query-common/src/main/java/org/apache/kylin/query/util/RestoreFromComputedColumn.java
@@ -237,7 +237,6 @@ public class RestoreFromComputedColumn implements
IPushDownConverter {
// however user query might use a different alias, say bc.x + ba.y
String ccExpression =
CalciteParser.replaceAliasInExpr(computedColumnDesc.getExpression(),
matchInfo.getAliasMap().inverse());
- // intend to handle situation like KE-15939
String replaceExpression = columnUsage.addAlias ? ccExpression + "
AS " + computedColumnDesc.getColumnName()
: ccExpression;
diff --git
a/src/query-service/src/main/java/org/apache/kylin/rest/service/QueryService.java
b/src/query-service/src/main/java/org/apache/kylin/rest/service/QueryService.java
index aae6a5edfb..31f6a9f353 100644
---
a/src/query-service/src/main/java/org/apache/kylin/rest/service/QueryService.java
+++
b/src/query-service/src/main/java/org/apache/kylin/rest/service/QueryService.java
@@ -721,9 +721,9 @@ public class QueryService extends BasicService implements
CacheSignatureQuerySup
try {
if (!sqlResponse.isPrepare() &&
QueryMetricsContext.isStarted()) {
val queryMetricsContext =
QueryMetricsContext.collect(QueryContext.current());
- // KE-35556 Set stored sql a structured format json string
+ // Set stored sql a structured format json string
queryMetricsContext.setSql(constructQueryHistorySqlText(sqlRequest,
sqlResponse, originalSql));
- // KE-36662 Using sql_pattern as normalized_sql storage
+ // Using sql_pattern as normalized_sql storage
String normalizedSql =
QueryContext.currentMetrics().getCorrectedSql();
queryMetricsContext.setSqlPattern(normalizedSql);
QueryHistoryScheduler queryHistoryScheduler =
QueryHistoryScheduler.getInstance();
diff --git
a/src/query-service/src/main/java/org/apache/kylin/rest/util/SparderUIUtil.java
b/src/query-service/src/main/java/org/apache/kylin/rest/util/SparderUIUtil.java
index 0f670be05d..e5b0ed0aa4 100644
---
a/src/query-service/src/main/java/org/apache/kylin/rest/util/SparderUIUtil.java
+++
b/src/query-service/src/main/java/org/apache/kylin/rest/util/SparderUIUtil.java
@@ -78,7 +78,6 @@ public class SparderUIUtil {
return;
}
- // KE-12678
proxyBase = "/proxy/" + ui.appId();
// reset
diff --git
a/src/query-service/src/test/java/org/apache/kylin/rest/metrics/QueryMetricsContextTest.java
b/src/query-service/src/test/java/org/apache/kylin/rest/metrics/QueryMetricsContextTest.java
index 715a9c81c2..0f25021b69 100644
---
a/src/query-service/src/test/java/org/apache/kylin/rest/metrics/QueryMetricsContextTest.java
+++
b/src/query-service/src/test/java/org/apache/kylin/rest/metrics/QueryMetricsContextTest.java
@@ -361,7 +361,6 @@ public class QueryMetricsContextTest extends
NLocalFileMetadataTestCase {
@Test
public void testWhenHitStorageCache() {
- //this is for https://olapio.atlassian.net/browse/KE-12573
final String origSql = "select * from test_parse_sql_pattern_error;";
final String massagedSql = "select * from
test_parse_sql_pattern_error";
final String sqlPattern = "SELECT *\n" + "FROM
\"TEST_PARSE_SQL_PATTERN_ERROR\"";
diff --git
a/src/query-service/src/test/java/org/apache/kylin/rest/service/QueryServiceTest.java
b/src/query-service/src/test/java/org/apache/kylin/rest/service/QueryServiceTest.java
index 1bcfbd7ed4..5209745448 100644
---
a/src/query-service/src/test/java/org/apache/kylin/rest/service/QueryServiceTest.java
+++
b/src/query-service/src/test/java/org/apache/kylin/rest/service/QueryServiceTest.java
@@ -1701,7 +1701,6 @@ public class QueryServiceTest extends
NLocalFileMetadataTestCase {
}
@Test
- //ref KE-12803
public void testDeepCopy() {
final List<TableMetaWithType> tableMetas =
queryService.getMetadataV2("default", null);
tableMetas.stream()
@@ -1732,7 +1731,6 @@ public class QueryServiceTest extends
NLocalFileMetadataTestCase {
}
@Test
- //reference KE-8052
public void testQueryWithScanBytesAndRows() {
long defaultValue = QueryContext.DEFAULT_NULL_SCANNED_DATA;
diff --git
a/src/query/src/main/java/org/apache/kylin/query/engine/AsyncQueryApplication.java
b/src/query/src/main/java/org/apache/kylin/query/engine/AsyncQueryApplication.java
index b36ccf4e92..24a977bee2 100644
---
a/src/query/src/main/java/org/apache/kylin/query/engine/AsyncQueryApplication.java
+++
b/src/query/src/main/java/org/apache/kylin/query/engine/AsyncQueryApplication.java
@@ -98,7 +98,7 @@ public class AsyncQueryApplication extends SparkApplication {
try {
QueryMetricsContext queryMetricsContext =
QueryMetricsContext.collect(queryContext);
queryMetricsContext.setSql(constructQueryHistorySqlText(queryParams,
queryContext.getUserSQL()));
- // KE-36662 Using sql_pattern as normalized_sql storage
+ // Using sql_pattern as normalized_sql storage
String normalizedSql =
QueryContext.currentMetrics().getCorrectedSql();
queryMetricsContext.setSqlPattern(normalizedSql);
diff --git
a/src/second-storage/clickhouse-it/src/test/java/io/kyligence/kap/secondstorage/abnormal/QueryRouteTest.java
b/src/second-storage/clickhouse-it/src/test/java/io/kyligence/kap/secondstorage/abnormal/QueryRouteTest.java
index 451945bc7b..5556db5d96 100644
---
a/src/second-storage/clickhouse-it/src/test/java/io/kyligence/kap/secondstorage/abnormal/QueryRouteTest.java
+++
b/src/second-storage/clickhouse-it/src/test/java/io/kyligence/kap/secondstorage/abnormal/QueryRouteTest.java
@@ -69,7 +69,6 @@ import io.kyligence.kap.secondstorage.test.SharedSparkSession;
* {@link QueryRouteTest} simulates the case where KE(resolving clickhouse
table schema on spark driver) or worker
* (executing query on spark executor) can not access ClickHouse.In such case,
we should use table index to answer query.
* <p/>
- * @see <a href="https://olapio.atlassian.net/browse/KE-28035";>KE-28035</a>
for details.
*/
public class QueryRouteTest {
diff --git
a/src/second-storage/clickhouse-it/src/test/java/io/kyligence/kap/secondstorage/abnormal/SecondaryCatalogTest.java
b/src/second-storage/clickhouse-it/src/test/java/io/kyligence/kap/secondstorage/abnormal/SecondaryCatalogTest.java
index 27b7c57530..4eedc1640f 100644
---
a/src/second-storage/clickhouse-it/src/test/java/io/kyligence/kap/secondstorage/abnormal/SecondaryCatalogTest.java
+++
b/src/second-storage/clickhouse-it/src/test/java/io/kyligence/kap/secondstorage/abnormal/SecondaryCatalogTest.java
@@ -70,7 +70,6 @@ public class SecondaryCatalogTest implements JobWaiter {
/**
* When set spark.sql.catalog.{queryCatalog}.url to an irrelevant
clickhouse JDBC URL, there is an bug before
- * KE-27650
*/
@Test
public void testSparkJdbcUrlNotExist() throws Exception {
diff --git
a/src/second-storage/core/src/main/java/org/apache/kylin/job/SecondStorageJobParamUtil.java
b/src/second-storage/core/src/main/java/org/apache/kylin/job/SecondStorageJobParamUtil.java
index 8dba249200..2545b64a22 100644
---
a/src/second-storage/core/src/main/java/org/apache/kylin/job/SecondStorageJobParamUtil.java
+++
b/src/second-storage/core/src/main/java/org/apache/kylin/job/SecondStorageJobParamUtil.java
@@ -78,7 +78,6 @@ public class SecondStorageJobParamUtil {
/**
* build delete layout table parameters
*
- * PRD_KE-34597 add index clean job
*
* @param project project name
* @param model model id
diff --git
a/src/spark-project/engine-spark/src/main/java/org/apache/kylin/engine/spark/merger/SparkJobMetadataMerger.java
b/src/spark-project/engine-spark/src/main/java/org/apache/kylin/engine/spark/merger/SparkJobMetadataMerger.java
index e41b48336e..da9d56e079 100644
---
a/src/spark-project/engine-spark/src/main/java/org/apache/kylin/engine/spark/merger/SparkJobMetadataMerger.java
+++
b/src/spark-project/engine-spark/src/main/java/org/apache/kylin/engine/spark/merger/SparkJobMetadataMerger.java
@@ -175,7 +175,7 @@ public abstract class SparkJobMetadataMerger extends
MetadataMerger {
item.forEach((k, v) -> //
merged.put(k, v + merged.getOrDefault(k, 0L))));
localSegment.setColumnSourceBytes(merged);
- // KE-18417 snapshot management.
+ // snapshot management.
localSegment.setLastBuildTime(newSegment.getLastBuildTime());
localSegment.setSourceBytesSize(newSegment.getSourceBytesSize());
localSegment.setLastBuildTime(lastBuildTime);
diff --git
a/src/spark-project/engine-spark/src/main/scala/org/apache/kylin/engine/spark/builder/DFDictionaryBuilder.scala
b/src/spark-project/engine-spark/src/main/scala/org/apache/kylin/engine/spark/builder/DFDictionaryBuilder.scala
index 78a5c2cd97..36437a7916 100644
---
a/src/spark-project/engine-spark/src/main/scala/org/apache/kylin/engine/spark/builder/DFDictionaryBuilder.scala
+++
b/src/spark-project/engine-spark/src/main/scala/org/apache/kylin/engine/spark/builder/DFDictionaryBuilder.scala
@@ -83,7 +83,6 @@ class DFDictionaryBuilder(
} finally lock.unlock()
}
- // Workaround: https://olapio.atlassian.net/browse/KE-41645
private[builder] def changeAQEConfig(isDictBuildFinished: Boolean = false) :
Boolean = {
if (!seg.getConfig.isGlobalDictAQEEnabled && !isDictBuildFinished) {
logInfo("Temporarily Close AQE for dict build job")
diff --git
a/src/spark-project/engine-spark/src/main/scala/org/apache/kylin/engine/spark/job/stage/build/FlatTableAndDictBase.scala
b/src/spark-project/engine-spark/src/main/scala/org/apache/kylin/engine/spark/job/stage/build/FlatTableAndDictBase.scala
index 3e94fbefee..aecced663b 100644
---
a/src/spark-project/engine-spark/src/main/scala/org/apache/kylin/engine/spark/job/stage/build/FlatTableAndDictBase.scala
+++
b/src/spark-project/engine-spark/src/main/scala/org/apache/kylin/engine/spark/job/stage/build/FlatTableAndDictBase.scala
@@ -358,7 +358,7 @@ abstract class FlatTableAndDictBase(private val jobContext:
SegmentJob,
DFBuilderHelper.checkPointSegment(dataSegment, (copied: NDataSegment) => {
copied.setFlatTableReady(true)
if (dataSegment.isFlatTableReady) {
- // KE-14714 if flat table is updated, there might be some data
inconsistency across indexes
+ // if flat table is updated, there might be some data inconsistency
across indexes
copied.setStatus(SegmentStatusEnum.WARNING)
}
})
diff --git
a/src/spark-project/engine-spark/src/test/java/org/apache/kylin/engine/spark/source/NSparkSourceTest.java
b/src/spark-project/engine-spark/src/test/java/org/apache/kylin/engine/spark/source/NSparkSourceTest.java
index 7a940be559..80f6445e97 100644
---
a/src/spark-project/engine-spark/src/test/java/org/apache/kylin/engine/spark/source/NSparkSourceTest.java
+++
b/src/spark-project/engine-spark/src/test/java/org/apache/kylin/engine/spark/source/NSparkSourceTest.java
@@ -57,7 +57,6 @@ public class NSparkSourceTest extends
NLocalWithSparkSessionTest {
}
/**
- * for the issue: https://olapio.atlassian.net/browse/KE-9497
*/
@Test
public void testGetSourceData() {
diff --git
a/src/spark-project/sparder/src/main/scala/org/apache/spark/sql/KylinSession.scala
b/src/spark-project/sparder/src/main/scala/org/apache/spark/sql/KylinSession.scala
index 2ebf1d4826..8ec136c9fe 100644
---
a/src/spark-project/sparder/src/main/scala/org/apache/spark/sql/KylinSession.scala
+++
b/src/spark-project/sparder/src/main/scala/org/apache/spark/sql/KylinSession.scala
@@ -181,7 +181,6 @@ object KylinSession extends Logging {
// maybe this is an existing SparkContext, update its SparkConf
which maybe used
// by SparkSession
- // KE-12678
if (sc.master.startsWith("yarn")) {
Unsafe.setProperty("spark.ui.proxyBase", "/proxy/" +
sc.applicationId)
}
diff --git
a/src/spark-project/sparder/src/test/scala/org/apache/spark/sql/udf/TimestampDiffTest.scala
b/src/spark-project/sparder/src/test/scala/org/apache/spark/sql/udf/TimestampDiffTest.scala
index c594160459..0f53f6322b 100644
---
a/src/spark-project/sparder/src/test/scala/org/apache/spark/sql/udf/TimestampDiffTest.scala
+++
b/src/spark-project/sparder/src/test/scala/org/apache/spark/sql/udf/TimestampDiffTest.scala
@@ -56,7 +56,6 @@ class TimestampDiffTest extends SparderBaseFunSuite with
SharedSparkSession with
// DAY
verifyResult("select timestampdiff('DAY', date'2016-02-01' ,
date'2016-02-01')", Seq("0"))
verifyResult("select timestampdiff('DAY', date'2016-02-01' ,
date'2016-02-02')", Seq("1"))
- // TODO KE-27654
// verifyResult("select timestampdiff('DAY', date'1977-04-20',
date'1987-08-02')", Seq("3756"))
// HOUR
diff --git
a/src/tool/src/main/java/org/apache/kylin/tool/kerberos/KerberosLoginTask.java
b/src/tool/src/main/java/org/apache/kylin/tool/kerberos/KerberosLoginTask.java
index 412f6ae658..5fc91a201a 100644
---
a/src/tool/src/main/java/org/apache/kylin/tool/kerberos/KerberosLoginTask.java
+++
b/src/tool/src/main/java/org/apache/kylin/tool/kerberos/KerberosLoginTask.java
@@ -34,7 +34,7 @@ import org.slf4j.LoggerFactory;
import org.apache.kylin.guava30.shaded.common.base.Preconditions;
/**
- * KE-Process kerberos long-running scenario.
+ * Process kerberos long-running scenario.
* @deprecated since 'ke-4.5.18.0'
* This implementation is no longer acceptable to fulfill in long-running
scenario.
* Use {@link DelegationTokenManager} instead.
