Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20856 )
Change subject: IMPALA-12676: Fix integer overflow in DatabaseTest ...................................................................... IMPALA-12676: Fix integer overflow in DatabaseTest Fixes an integer overflow issue in the DatabaseTest class in the internal-server-test.cc file by switching from a signed to an unsigned int. Additional comments and assertions were also added to prevent future integer overflows. Change-Id: I3689185aa8676203226cc447585703e784627102 Reviewed-on: http://gerrit.cloudera.org:8080/20856 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Michael Smith <[email protected]> --- M be/src/service/internal-server-test.cc 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved Michael Smith: Verified -- To view, visit http://gerrit.cloudera.org:8080/20856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3689185aa8676203226cc447585703e784627102 Gerrit-Change-Number: 20856 Gerrit-PatchSet: 5 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
