Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/22298 )
Change subject: WIP: refactor Frontend part 1 ...................................................................... Patch Set 1: (58 comments) gerrit-auto-critic failed. You can reproduce it locally using command: python3 bin/jenkins/critique-gerrit-review.py --dryrun To run it, you might need a virtual env with Python3's venv installed. http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/AnalysisDriverImpl.java File fe/src/main/java/org/apache/impala/analysis/AnalysisDriverImpl.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/AnalysisDriverImpl.java@63 PS1, Line 63: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/AnalysisResult.java File fe/src/main/java/org/apache/impala/analysis/AnalysisResult.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/AnalysisResult.java@63 PS1, Line 63: protected static final Logger LOG = LoggerFactory.getLogger(AnalysisResult.class.getName()); line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java File fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java@70 PS1, Line 70: protected CreateFunctionStmtBase(Set<StatementType> stmtTypes, FunctionName fnName, FunctionArgs args, line too long (104 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/CreateOrAlterViewStmtBase.java File fe/src/main/java/org/apache/impala/analysis/CreateOrAlterViewStmtBase.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/CreateOrAlterViewStmtBase.java@80 PS1, Line 80: public CreateOrAlterViewStmtBase(Set<StatementType> stmtTypes, boolean ifNotExists, TableName tableName, line too long (106 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/DeleteStmt.java File fe/src/main/java/org/apache/impala/analysis/DeleteStmt.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/DeleteStmt.java@48 PS1, Line 48: public class DeleteStmt extends ModifyStmt line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/DescribeDbStmt.java File fe/src/main/java/org/apache/impala/analysis/DescribeDbStmt.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/DescribeDbStmt.java@112 PS1, Line 112: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/ModifyStmt.java File fe/src/main/java/org/apache/impala/analysis/ModifyStmt.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/ModifyStmt.java@92 PS1, Line 92: public ModifyStmt(Set<StatementType> stmtTypes, List<String> targetTablePath, FromClause fromClause, line too long (102 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/QueryStmt.java File fe/src/main/java/org/apache/impala/analysis/QueryStmt.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/QueryStmt.java@107 PS1, Line 107: QueryStmt(Set<StatementType> stmtTypes, List<OrderByElement> orderByElements, LimitElement limitElement) { line too long (108 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/ShowTablesOrViewsStmt.java File fe/src/main/java/org/apache/impala/analysis/ShowTablesOrViewsStmt.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/ShowTablesOrViewsStmt.java@65 PS1, Line 65: public ShowTablesOrViewsStmt(Set<StatementType> stmtType, String database, String pattern) { line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/StatementBase.java File fe/src/main/java/org/apache/impala/analysis/StatementBase.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/StatementBase.java@301 PS1, Line 301: public TCatalogOpRequest getTCatalogOpRequest(TQueryOptions queryOptions) line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/UpdateStmt.java File fe/src/main/java/org/apache/impala/analysis/UpdateStmt.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/analysis/UpdateStmt.java@69 PS1, Line 69: StatementType.DML); line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/planner/Planner.java File fe/src/main/java/org/apache/impala/planner/Planner.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/planner/Planner.java@357 PS1, Line 357: if (ctx_.getAnalysisResult().getStmt().isExplain() || RuntimeEnv.INSTANCE.isTestEnv()) { line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/planner/PlannerContext.java File fe/src/main/java/org/apache/impala/planner/PlannerContext.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/planner/PlannerContext.java@113 PS1, Line 113: public boolean isOptimize() { return analysisResult_.getStmt().isStmtType(StatementType.OPTIMIZE);} line too long (101 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/planner/SingleNodePlannerImpl.java File fe/src/main/java/org/apache/impala/planner/SingleNodePlannerImpl.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/planner/SingleNodePlannerImpl.java@533 PS1, Line 533: PlanNode result = createJoinPlan(analyzer, candidate.first, parentRefPlans, subplanRefs); line too long (95 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/planner/SingleNodePlannerImpl.java@1151 PS1, Line 1151: TupleDescriptor tupleDesc = createResultTupleDescriptor(selectStmt, "union", analyzer); line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/service/BETestJniFrontend.java File fe/src/main/java/org/apache/impala/service/BETestJniFrontend.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/service/BETestJniFrontend.java@258 PS1, Line 258: public TestTransactionContextBuilder(BackendConfig backendConfig) throws ImpalaException { line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/service/CoordinatorFrontendContextBuilder.java File fe/src/main/java/org/apache/impala/service/CoordinatorFrontendContextBuilder.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/service/CoordinatorFrontendContextBuilder.java@251 PS1, Line 251: private AuthorizationFactory authzFactory_; line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/service/CoordinatorFrontendContextBuilder.java@256 PS1, Line 256: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/service/CoordinatorFrontendContextBuilder.java@269 PS1, Line 269: public CoordinatorFrontendContextBuilder(BackendConfig backendConfig) throws ImpalaException { line too long (96 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/service/FrontendContext.java File fe/src/main/java/org/apache/impala/service/FrontendContext.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/service/FrontendContext.java@281 PS1, Line 281: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/ParsedStatementImpl.java File fe/src/main/java/org/apache/impala/stmtdriver/ParsedStatementImpl.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/ParsedStatementImpl.java@85 PS1, Line 85: @Override line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/PlanCtx.java File fe/src/main/java/org/apache/impala/stmtdriver/PlanCtx.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/PlanCtx.java@423 PS1, Line 423: "org.apache.impala.calcite.service.CalciteParsedStatementCreator").newInstance(); line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/PlanCtx.java@432 PS1, Line 432: } line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/PlannerDriver.java File fe/src/main/java/org/apache/impala/stmtdriver/PlannerDriver.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/PlannerDriver.java@422 PS1, Line 422: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/PlannerDriver.java@445 PS1, Line 445: TExecRequest result = analysisResult.getStmt().isStmtType(StatementType.CREATE_TABLE_AS_SELECT) line too long (99 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/StmtDriver.java File fe/src/main/java/org/apache/impala/stmtdriver/StmtDriver.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/StmtDriver.java@685 PS1, Line 685: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/Transaction.java File fe/src/main/java/org/apache/impala/stmtdriver/Transaction.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/Transaction.java@253 PS1, Line 253: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/Transaction.java@511 PS1, Line 511: !(analysisResult.getStmt().isStmtType(StatementType.INSERT) || line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/Transaction.java@537 PS1, Line 537: if (analysisResult.getStmt().isExplain() || !queryOptions.isEnable_kudu_transaction()) { line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/TransactionContext.java File fe/src/main/java/org/apache/impala/stmtdriver/TransactionContext.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/main/java/org/apache/impala/stmtdriver/TransactionContext.java@278 PS1, Line 278: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/analysis/AuditingTest.java File fe/src/test/java/org/apache/impala/analysis/AuditingTest.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/analysis/AuditingTest.java@411 PS1, Line 411: TransactionContext.Builder transactionContextBuilder = new TransactionContext.Builder( line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/analysis/ParserTest.java File fe/src/test/java/org/apache/impala/analysis/ParserTest.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/analysis/ParserTest.java@372 PS1, Line 372: CreateTableAsSelectStmt ctasStmt = (CreateTableAsSelectStmt) parsedStmt.getTopLevelNode(); line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/analysis/ParserTest.java@1719 PS1, Line 1719: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java File fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java@63 PS1, Line 63: TransactionContext.Builder transactionContextBuilder = new TransactionContext.Builder( line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java@81 PS1, Line 81: TransactionContext.Builder transactionContextBuilder = new TransactionContext.Builder( line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java@96 PS1, Line 96: TransactionContext.Builder transactionContextBuilder = new TransactionContext.Builder( line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/authorization/AuthorizationTestBase.java File fe/src/test/java/org/apache/impala/authorization/AuthorizationTestBase.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/authorization/AuthorizationTestBase.java@138 PS1, Line 138: TransactionContext.Builder transactionContextBuilder = new TransactionContext.Builder( line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/common/FrontendFixture.java File fe/src/test/java/org/apache/impala/common/FrontendFixture.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/common/FrontendFixture.java@270 PS1, Line 270: QueryStmt viewStmt = (QueryStmt) parseStmt(createViewStmt.getInlineViewDef()).getTopLevelNode(); line too long (100 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/common/FrontendFixture.java@377 PS1, Line 377: TestParsedStatement parsedStmt = new TestParsedStatement(stmt, ctx.getQueryCtx().client_request.query_options); line too long (115 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/common/FrontendTestBase.java File fe/src/test/java/org/apache/impala/common/FrontendTestBase.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/common/FrontendTestBase.java@85 PS1, Line 85: protected static final Logger LOG = LoggerFactory.getLogger(FrontendTestBase.class.getName()); line too long (96 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/common/TestParsedStatement.java File fe/src/test/java/org/apache/impala/common/TestParsedStatement.java: http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/common/TestParsedStatement.java@58 PS1, Line 58: protected static final Logger LOG = LoggerFactory.getLogger(TestParsedStatement.class.getName()); line too long (99 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/common/TestParsedStatement.java@75 PS1, Line 75: stmt_ = Parser.parse(stmtSql); line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/fe/src/test/java/org/apache/impala/common/TestParsedStatement.java@83 PS1, Line 83: @Override line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteAnalysisDriver.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteAnalysisDriver.java: http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteAnalysisDriver.java@74 PS1, Line 74: public class CalciteAnalysisDriver implements AnalysisDriver { line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteAnalysisDriver.java@75 PS1, Line 75: protected static final Logger LOG = LoggerFactory.getLogger(CalciteAnalysisDriver.class.getName()); line too long (101 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteAnalysisDriver.java@106 PS1, Line 106: } line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteAnalysisDriver.java@116 PS1, Line 116: CalciteMetadataHandler.populateCalciteSchema(reader_, ctx.getCatalog(), line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteAnalysisDriver.java@118 PS1, Line 118: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteAnalysisDriver.java@152 PS1, Line 152: } line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteParsedStatement.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteParsedStatement.java: http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteParsedStatement.java@46 PS1, Line 46: protected static final Logger LOG = LoggerFactory.getLogger(CalciteParsedStatement.class.getName()); line too long (102 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteParsedStatement.java@104 PS1, Line 104: throw new RuntimeException("This needs reimplementing, impossible for Calcite to do here."); line too long (96 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteParsedStatement.java@112 PS1, Line 112: @Override line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteParsedStatementCreator.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteParsedStatementCreator.java: http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteParsedStatementCreator.java@49 PS1, Line 49: line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalcitePhysPlanCreator.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalcitePhysPlanCreator.java: http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalcitePhysPlanCreator.java@60 PS1, Line 60: this.plannerContext_ = line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteSingleNodePlanner.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteSingleNodePlanner.java: http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteSingleNodePlanner.java@120 PS1, Line 120: protected static final Logger LOG = LoggerFactory.getLogger(CalciteSingleNodePlanner.class.getName()); line too long (104 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteSingleNodePlanner.java@125 PS1, Line 125: public CalciteSingleNodePlanner(PlannerContext ctx, CalciteAnalysisDriver analysisDriver) { line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/java/external-frontend/src/main/java/org/apache/impala/external/TestJniFrontend.java File java/external-frontend/src/main/java/org/apache/impala/external/TestJniFrontend.java: http://gerrit.cloudera.org:8080/#/c/22298/1/java/external-frontend/src/main/java/org/apache/impala/external/TestJniFrontend.java@88 PS1, Line 88: public TestTransactionContextBuilder(BackendConfig backendConfig) throws ImpalaException { line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/22298/1/java/test-frontend/src/main/java/org/apache/impala/service/BETestJniFrontend.java File java/test-frontend/src/main/java/org/apache/impala/service/BETestJniFrontend.java: http://gerrit.cloudera.org:8080/#/c/22298/1/java/test-frontend/src/main/java/org/apache/impala/service/BETestJniFrontend.java@257 PS1, Line 257: public TestTransactionContextBuilder(BackendConfig backendConfig) throws ImpalaException { line too long (94 > 90) -- To view, visit http://gerrit.cloudera.org:8080/22298 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2617659e5d4da394cb4fda325f40ac93b12adfb3 Gerrit-Change-Number: 22298 Gerrit-PatchSet: 1 Gerrit-Owner: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]> Gerrit-Comment-Date: Sat, 04 Jan 2025 00:01:15 +0000 Gerrit-HasComments: Yes
