Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12660 )
Change subject: IMPALA-8250: Clean up JNI warnings. ...................................................................... Patch Set 3: > "I've kicked off a suite of our tests with code coverage on. I'm > also going to kick off a set of tests with the JNI checking and > 'exhaustive' tests. Maybe there's coverage, but we're not running > the queries against HBase." > > Did anything new come up in this? > Nothing new. I'm still chasing a sequence of these calls which lead to CallObjectMethod() not having its exception checked... checked_jni_CallVoidMethodV checked_jni_ExceptionOccurred checked_jni_ExceptionClear checked_jni_GetObjectClass checked_jni_FindClass checked_jni_GetMethodID checked_jni_CallObjectMethod I can trigger this with set mem_limit=50m;use functional;insert into table alltypesinsert partition (year, month) /* +noclustered */ select at1.id, at1.bool_col, at1.tinyint_col, at1.smallint_col, at1.int_col, at1.bigint_col, at1.float_col, at1.double_col, at1.date_string_col, at1.string_col, at1.timestamp_col, at1.year, at2.id as month from functional.alltypes at1, functional.alltypes at2; when the sink gets closed after a failure. I don't know where this stack of calls is at this point. I'll keep looking, but I don't want to block on it. -- To view, visit http://gerrit.cloudera.org:8080/12660 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idd1709f749a764c1d947704bc64306493863b45f Gerrit-Change-Number: 12660 Gerrit-PatchSet: 3 Gerrit-Owner: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Sahil Takiar <stak...@cloudera.com> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Comment-Date: Thu, 07 Mar 2019 23:08:14 +0000 Gerrit-HasComments: No