Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22955 )
Change subject: IMPALA-14104: Fix TestDecimalFuzz on Python 3 ...................................................................... IMPALA-14104: Fix TestDecimalFuzz on Python 3 TestDecimalFuzz uses division to calculate the number of iterations for certain tests. On Python 3, division produces a float and range() will not take a float as an argument. In theory, the "from __future__ import division" was supposed to produce the same behavior on Python 2 and 3, but in practice, the "from builtins import range" allows a float argument to range() on Python 2 but not Python 3. This fixes the issue by explicitly casting to an integer. Testing: - Ran TestDecimalFuzz with Python 3 Change-Id: I4cd4daecde690bf41a4e412c02c23cbb6ae5a14c Reviewed-on: http://gerrit.cloudera.org:8080/22955 Reviewed-by: Riza Suminto <riza.sumi...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M tests/query_test/test_decimal_fuzz.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Riza Suminto: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/22955 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I4cd4daecde690bf41a4e412c02c23cbb6ae5a14c Gerrit-Change-Number: 22955 Gerrit-PatchSet: 3 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
