Laszlo Gaal has posted comments on this change. ( http://gerrit.cloudera.org:8080/12894 )
Change subject: IMPALA-8380: Bump Postgres JDBC driver version to 9.4 ...................................................................... Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/12894/3//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/12894/3//COMMIT_MSG@7 PS3, Line 7: 9.4 Update the headline :) http://gerrit.cloudera.org:8080/#/c/12894/1/bin/impala-config.sh File bin/impala-config.sh: http://gerrit.cloudera.org:8080/#/c/12894/1/bin/impala-config.sh@567 PS1, Line 567: jar" > I'm curious why we chose the one with jre7 instead of the default one. I was probably overly cautious, thinking about the remote chance of this change being backported to cdh5 (e.g. a critical bugfix comes along upstream, which depends on the new driver version). This will probably not happen; and if it does, we can still pick a different, more compatible driver version at that time. @Lars: if'n'when you bump the version to 42.something, feel free to move away from jre7 as well. -- To view, visit http://gerrit.cloudera.org:8080/12894 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ica5423c18a9f8346dda7dae617b1764638b57b6c Gerrit-Change-Number: 12894 Gerrit-PatchSet: 3 Gerrit-Owner: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Comment-Date: Thu, 04 Apr 2019 14:51:33 +0000 Gerrit-HasComments: Yes
