Fucun Chu has posted comments on this change. ( http://gerrit.cloudera.org:8080/17793 )
Change subject: IMPALA-10870: Add Apache Hive 3.1.2 to the minicluster ...................................................................... Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/17793/3/bin/bootstrap_toolchain.py File bin/bootstrap_toolchain.py: http://gerrit.cloudera.org:8080/#/c/17793/3/bin/bootstrap_toolchain.py@324 PS3, Line 324: class ApacheComponent(EnvVersionedPackage): > flake8: E302 expected 2 blank lines, found 1 Done http://gerrit.cloudera.org:8080/#/c/17793/3/bin/bootstrap_toolchain.py@326 PS3, Line 326: > flake8: E251 unexpected spaces around keyword / parameter equals Done http://gerrit.cloudera.org:8080/#/c/17793/3/bin/impala-config.sh File bin/impala-config.sh: http://gerrit.cloudera.org:8080/#/c/17793/3/bin/impala-config.sh@261 PS3, Line 261: # When USE_APACHE_HIVE is set we use the apache hive version to build as well as deploy in > line too long (92 > 90) Done -- To view, visit http://gerrit.cloudera.org:8080/17793 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1978909589ecacb15d32d874e97f050a85adf1f6 Gerrit-Change-Number: 17793 Gerrit-PatchSet: 3 Gerrit-Owner: Fucun Chu <chufu...@hotmail.com> Gerrit-Reviewer: Fucun Chu <chufu...@hotmail.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Comment-Date: Sat, 21 Aug 2021 04:22:13 +0000 Gerrit-HasComments: Yes