Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10746 )
Change subject: IMPALA-7150: fixes jni frame scope ...................................................................... Patch Set 1: (1 comment) The change seems safe and the theory makes sense. Were you able to reproduce the crash reliably enough to confirm that this fixed it? http://gerrit.cloudera.org:8080/#/c/10746/1/be/src/exprs/hive-udf-call.cc File be/src/exprs/hive-udf-call.cc: http://gerrit.cloudera.org:8080/#/c/10746/1/be/src/exprs/hive-udf-call.cc@202 PS1, Line 202: trailing whitespace. -- To view, visit http://gerrit.cloudera.org:8080/10746 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I38d482dde6ed7f8857ae565bf62a8e923ffb0ae9 Gerrit-Change-Number: 10746 Gerrit-PatchSet: 1 Gerrit-Owner: Vuk Ercegovac <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Mon, 18 Jun 2018 15:57:16 +0000 Gerrit-HasComments: Yes
