Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/9626 )
Change subject: IMPALA-6488: removes use-after-free bug in lib_cache ...................................................................... Patch Set 2: (5 comments) thanks for the review! http://gerrit.cloudera.org:8080/#/c/9626/2/tests/query_test/test_udfs.py File tests/query_test/test_udfs.py: http://gerrit.cloudera.org:8080/#/c/9626/2/tests/query_test/test_udfs.py@328 PS2, Line 328: setup = create_fn_to_use.format(unique_database, '/test-warehouse/libTestUdfs.so') > Nit: this could probably go before the try block. Done http://gerrit.cloudera.org:8080/#/c/9626/2/tests/query_test/test_udfs.py@331 PS2, Line 331: print e > You can print/log any extra info you like here, but why not then raise the Done http://gerrit.cloudera.org:8080/#/c/9626/2/tests/query_test/test_udfs.py@341 PS2, Line 341: except Exception as e: errors.append(e) > PEP-8 discourages inlining like this. Break after the colon (throughout). Done http://gerrit.cloudera.org:8080/#/c/9626/2/tests/query_test/test_udfs.py@351 PS2, Line 351: except Exception as e: errors.append(e) > Break errors.append(e) onto a new line. Done http://gerrit.cloudera.org:8080/#/c/9626/2/tests/query_test/test_udfs.py@358 PS2, Line 358: for i in range(0, num_uses): > Nit: a bit pedantic, but xrange is favored over range in python 2.x. (In fa Done -- To view, visit http://gerrit.cloudera.org:8080/9626 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1f178a2114cb3969dbb920949ddff4e8220b742e Gerrit-Change-Number: 9626 Gerrit-PatchSet: 2 Gerrit-Owner: Vuk Ercegovac <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]> Gerrit-Comment-Date: Wed, 14 Mar 2018 21:47:40 +0000 Gerrit-HasComments: Yes
