Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11881 )
Change subject: IMPALA-7807: Analysis test fixture ...................................................................... Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/11881/4/fe/src/test/java/org/apache/impala/analysis/QueryFixture.java File fe/src/test/java/org/apache/impala/analysis/QueryFixture.java: http://gerrit.cloudera.org:8080/#/c/11881/4/fe/src/test/java/org/apache/impala/analysis/QueryFixture.java@72 PS4, Line 72: parsedStmt_ = this.analysisFixture_.frontend_.parse(stmt_, analysCtx_.getQueryOptions()); line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/11881/4/fe/src/test/java/org/apache/impala/analysis/QueryFixture.java@118 PS4, Line 118: new StmtMetadataLoader(this.analysisFixture_.frontend_, analysCtx_.getQueryCtx().session.database, null); line too long (112 > 90) -- To view, visit http://gerrit.cloudera.org:8080/11881 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id69c99b284960f16394f61072d38dd00269bc10c Gerrit-Change-Number: 11881 Gerrit-PatchSet: 4 Gerrit-Owner: Paul Rogers <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Thu, 15 Nov 2018 22:15:47 +0000 Gerrit-HasComments: Yes
