Skip to content

Commit 34132e9

Browse files
committed
Refactor: remove stale comments
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
1 parent 927dafd commit 34132e9

3 files changed

Lines changed: 0 additions & 3 deletions

File tree

datasources/src/main/java/org/opensearch/sql/datasources/service/DataSourceServiceImpl.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010

1111
import java.util.*;
1212
import java.util.stream.Collectors;
13-
1413
import org.opensearch.sql.datasource.DataSourceService;
1514
import org.opensearch.sql.datasource.RequestContext;
1615
import org.opensearch.sql.datasource.model.DataSource;

integ-test/src/test/java/org/opensearch/sql/calcite/standalone/CalcitePPLIntegTestCase.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -374,7 +374,6 @@ public QueryPlanFactory queryPlanFactory(ExecutionEngine executionEngine) {
374374
new Analyzer(
375375
new ExpressionAnalyzer(functionRepository), dataSourceService, functionRepository);
376376
Planner planner = new Planner(LogicalPlanOptimizer.create());
377-
// NodeClient is not used in integration test, so we pass null
378377
QueryService queryService =
379378
new QueryService(analyzer, executionEngine, planner, dataSourceService, settings);
380379
return new QueryPlanFactory(queryService);

ppl/src/test/java/org/opensearch/sql/ppl/calcite/CalcitePPLAbstractTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,6 @@ protected CalcitePlanContext createBuilderContext() {
7373
/** Creates a CalcitePlanContext with transformed config. */
7474
private CalcitePlanContext createBuilderContext(UnaryOperator<RelBuilder.Config> transform) {
7575
config.context(Contexts.of(transform.apply(RelBuilder.Config.DEFAULT)));
76-
// NodeClient is not used in this test, so we pass null
7776
return CalcitePlanContext.create(config.build(), PPL);
7877
}
7978

0 commit comments

Comments
 (0)