Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/17122 )
Change subject: IMPALA-10549: Register transactions from external frontend DML ...................................................................... Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/17122/5/be/src/service/impala-server.cc File be/src/service/impala-server.cc: http://gerrit.cloudera.org:8080/#/c/17122/5/be/src/service/impala-server.cc@1212 PS5, Line 1212: RETURN_IF_ERROR(exec_env_->frontend()->addTransaction(exec_request->query_exec_request.query_ctx)); line too long (107 > 90) http://gerrit.cloudera.org:8080/#/c/17122/5/fe/src/main/java/org/apache/impala/service/JniFrontend.java File fe/src/main/java/org/apache/impala/service/JniFrontend.java: http://gerrit.cloudera.org:8080/#/c/17122/5/fe/src/main/java/org/apache/impala/service/JniFrontend.java@695 PS5, Line 695: public void addTransaction(byte[] thriftQueryContext) throws TransactionException, ImpalaException { line too long (102 > 90) -- To view, visit http://gerrit.cloudera.org:8080/17122 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia8863b8d9d281a5d164f10de9c5ee52cf3be63db Gerrit-Change-Number: 17122 Gerrit-PatchSet: 5 Gerrit-Owner: Kurt Deschler <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Kurt Deschler <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Comment-Date: Fri, 26 Feb 2021 14:35:36 +0000 Gerrit-HasComments: Yes
