From 615a9a75ace759d913ab12d20c6a68f33d483174 Mon Sep 17 00:00:00 2001 From: Xiang Fu Date: Tue, 4 Apr 2023 14:49:25 -0700 Subject: [PATCH] fix checkstyle --- pinot-compatibility-verifier/pom.xml | 16 ++++++ .../pinot-spark-2-connector/pom.xml | 3 -- .../pinot-spark-3-connector/pom.xml | 3 -- pinot-connectors/pinot-spark-common/pom.xml | 3 -- .../common/CaseInsensitiveStringMap.java | 50 +++++++++---------- pinot-integration-test-base/pom.xml | 2 - .../integration/tests/QueryGenerator.java | 21 ++++---- 7 files changed, 52 insertions(+), 46 deletions(-) diff --git a/pinot-compatibility-verifier/pom.xml b/pinot-compatibility-verifier/pom.xml index a9d625f953d4..07754e386d94 100644 --- a/pinot-compatibility-verifier/pom.xml +++ b/pinot-compatibility-verifier/pom.xml @@ -84,6 +84,10 @@ net.minidev json-smart + + com.typesafe.netty + netty-reactive-streams + @@ -96,12 +100,24 @@ org.testng testng runtime + + + com.beust + jcommander + + org.apache.pinot pinot-controller ${project.version} test-jar + + + jakarta.activation + jakarta.activation-api + + diff --git a/pinot-connectors/pinot-spark-2-connector/pom.xml b/pinot-connectors/pinot-spark-2-connector/pom.xml index 7de3fb9f56b0..122a41987874 100644 --- a/pinot-connectors/pinot-spark-2-connector/pom.xml +++ b/pinot-connectors/pinot-spark-2-connector/pom.xml @@ -38,9 +38,6 @@ 1.3.0 3.1.1 org.apache.pinot.\$internal - - - false diff --git a/pinot-connectors/pinot-spark-3-connector/pom.xml b/pinot-connectors/pinot-spark-3-connector/pom.xml index fc5b9bf6d40d..9c2766f8adf6 100644 --- a/pinot-connectors/pinot-spark-3-connector/pom.xml +++ b/pinot-connectors/pinot-spark-3-connector/pom.xml @@ -37,9 +37,6 @@ 4.8 3.1.1 org.apache.pinot.\$internal - - - false diff --git a/pinot-connectors/pinot-spark-common/pom.xml b/pinot-connectors/pinot-spark-common/pom.xml index bc4196b071ec..e5cc8b7244f5 100644 --- a/pinot-connectors/pinot-spark-common/pom.xml +++ b/pinot-connectors/pinot-spark-common/pom.xml @@ -37,9 +37,6 @@ 2.8 1.3.0 3.1.1 - - - false diff --git a/pinot-connectors/pinot-spark-common/src/main/java/org/apache/pinot/connector/spark/common/CaseInsensitiveStringMap.java b/pinot-connectors/pinot-spark-common/src/main/java/org/apache/pinot/connector/spark/common/CaseInsensitiveStringMap.java index 788befe4270e..4c8d804a6b6b 100644 --- a/pinot-connectors/pinot-spark-common/src/main/java/org/apache/pinot/connector/spark/common/CaseInsensitiveStringMap.java +++ b/pinot-connectors/pinot-spark-common/src/main/java/org/apache/pinot/connector/spark/common/CaseInsensitiveStringMap.java @@ -41,39 +41,39 @@ * keys converted to lower case. This map doesn't allow null key. */ public class CaseInsensitiveStringMap implements Map { - private final Logger logger = LoggerFactory.getLogger(CaseInsensitiveStringMap.class); + private final Logger _logger = LoggerFactory.getLogger(CaseInsensitiveStringMap.class); - private String unsupportedOperationMsg = "CaseInsensitiveStringMap is read-only."; + private String _unsupportedOperationMsg = "CaseInsensitiveStringMap is read-only."; public static CaseInsensitiveStringMap empty() { return new CaseInsensitiveStringMap(new HashMap<>(0)); } - private final Map original; + private final Map _original; - private final Map delegate; + private final Map _delegate; public CaseInsensitiveStringMap(Map originalMap) { - original = new HashMap<>(originalMap); - delegate = new HashMap<>(originalMap.size()); + _original = new HashMap<>(originalMap); + _delegate = new HashMap<>(originalMap.size()); for (Map.Entry entry : originalMap.entrySet()) { String key = toLowerCase(entry.getKey()); - if (delegate.containsKey(key)) { - logger.warn("Converting duplicated key " + entry.getKey() + - " into CaseInsensitiveStringMap."); + if (_delegate.containsKey(key)) { + _logger.warn("Converting duplicated key " + entry.getKey() + + " into CaseInsensitiveStringMap."); } - delegate.put(key, entry.getValue()); + _delegate.put(key, entry.getValue()); } } @Override public int size() { - return delegate.size(); + return _delegate.size(); } @Override public boolean isEmpty() { - return delegate.isEmpty(); + return _delegate.isEmpty(); } private String toLowerCase(Object key) { @@ -82,52 +82,52 @@ private String toLowerCase(Object key) { @Override public boolean containsKey(Object key) { - return delegate.containsKey(toLowerCase(key)); + return _delegate.containsKey(toLowerCase(key)); } @Override public boolean containsValue(Object value) { - return delegate.containsValue(value); + return _delegate.containsValue(value); } @Override public String get(Object key) { - return delegate.get(toLowerCase(key)); + return _delegate.get(toLowerCase(key)); } @Override public String put(String key, String value) { - throw new UnsupportedOperationException(unsupportedOperationMsg); + throw new UnsupportedOperationException(_unsupportedOperationMsg); } @Override public String remove(Object key) { - throw new UnsupportedOperationException(unsupportedOperationMsg); + throw new UnsupportedOperationException(_unsupportedOperationMsg); } @Override public void putAll(Map m) { - throw new UnsupportedOperationException(unsupportedOperationMsg); + throw new UnsupportedOperationException(_unsupportedOperationMsg); } @Override public void clear() { - throw new UnsupportedOperationException(unsupportedOperationMsg); + throw new UnsupportedOperationException(_unsupportedOperationMsg); } @Override public Set keySet() { - return delegate.keySet(); + return _delegate.keySet(); } @Override public Collection values() { - return delegate.values(); + return _delegate.values(); } @Override public Set> entrySet() { - return delegate.entrySet(); + return _delegate.entrySet(); } /** @@ -179,7 +179,7 @@ public double getDouble(String key, double defaultValue) { * Returns the original case-sensitive map. */ public Map asCaseSensitiveMap() { - return Collections.unmodifiableMap(original); + return Collections.unmodifiableMap(_original); } @Override @@ -191,11 +191,11 @@ public boolean equals(Object o) { return false; } CaseInsensitiveStringMap that = (CaseInsensitiveStringMap) o; - return delegate.equals(that.delegate); + return _delegate.equals(that._delegate); } @Override public int hashCode() { - return Objects.hash(delegate); + return Objects.hash(_delegate); } } diff --git a/pinot-integration-test-base/pom.xml b/pinot-integration-test-base/pom.xml index 510dfe1cde2a..936baeed4073 100644 --- a/pinot-integration-test-base/pom.xml +++ b/pinot-integration-test-base/pom.xml @@ -38,8 +38,6 @@ 0.2.11 3.0.0 2.14.28 - - false diff --git a/pinot-integration-test-base/src/test/java/org/apache/pinot/integration/tests/QueryGenerator.java b/pinot-integration-test-base/src/test/java/org/apache/pinot/integration/tests/QueryGenerator.java index 83abdc52baca..1778b4371c14 100644 --- a/pinot-integration-test-base/src/test/java/org/apache/pinot/integration/tests/QueryGenerator.java +++ b/pinot-integration-test-base/src/test/java/org/apache/pinot/integration/tests/QueryGenerator.java @@ -508,25 +508,26 @@ public String generateH2Query() { List h2AggregateColumnAndFunctions = new ArrayList<>(); for (String aggregateColumnAndFunction : _aggregateColumnsAndFunctions) { String h2AggregateColumnAndFunction; - if (!aggregateColumnAndFunction.equals("COUNT(*)")) { - aggregateColumnAndFunction = aggregateColumnAndFunction.replace("(", "(`").replace(")", "`)"); + String pinotAggregateFunction = aggregateColumnAndFunction; + if (!pinotAggregateFunction.equals("COUNT(*)")) { + pinotAggregateFunction = pinotAggregateFunction.replace("(", "(`").replace(")", "`)"); } - if (!aggregateColumnAndFunction.contains("(")) { - aggregateColumnAndFunction = String.format("`%s`", aggregateColumnAndFunction); + if (!pinotAggregateFunction.contains("(")) { + pinotAggregateFunction = String.format("`%s`", pinotAggregateFunction); } // Make 'AVG' and - if (aggregateColumnAndFunction.startsWith("DISTINCTCOUNT(")) { + if (pinotAggregateFunction.startsWith("DISTINCTCOUNT(")) { // make 'DISTINCTCOUNT(..)' compatible with H2 SQL query using 'COUNT(DISTINCT(..)' - h2AggregateColumnAndFunction = aggregateColumnAndFunction.replace("DISTINCTCOUNT(", "COUNT(DISTINCT "); - } else if (AGGREGATION_FUNCTIONS.contains(aggregateColumnAndFunction.substring(0, 3))) { + h2AggregateColumnAndFunction = pinotAggregateFunction.replace("DISTINCTCOUNT(", "COUNT(DISTINCT "); + } else if (AGGREGATION_FUNCTIONS.contains(pinotAggregateFunction.substring(0, 3))) { // make AGG functions (SUM, MIN, MAX, AVG) compatible with H2 SQL query. // this is because Pinot queries casts all to double before doing aggregation - String aggFunctionName = aggregateColumnAndFunction.substring(0, 3); - h2AggregateColumnAndFunction = aggregateColumnAndFunction + String aggFunctionName = pinotAggregateFunction.substring(0, 3); + h2AggregateColumnAndFunction = pinotAggregateFunction .replace(aggFunctionName + "(", aggFunctionName + "(CAST(") .replace(")", " AS DOUBLE))"); } else { - h2AggregateColumnAndFunction = aggregateColumnAndFunction; + h2AggregateColumnAndFunction = pinotAggregateFunction; } h2AggregateColumnAndFunctions.add(h2AggregateColumnAndFunction); }