Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23083 )
Change subject: IMPALA-14157: Fix string representation of binary columns for Python 3 ...................................................................... IMPALA-14157: Fix string representation of binary columns for Python 3 When running tests with Python 3, several tests are failing when comparing the results for binary columns. Python 3 represents binary columns as bytes. When this gets converted to a string, it gets wrapped with a b'...', which causes difference from the expected value (e.g. b'whatever' vs whatever). This adds decoding logic to instead decode the bytes to a string without the added differences. This uses 'backslashdecode' to avoid throwing an error for invalid Unicode. Testing: - Ran several tests that use binary results with Python 2 and Python 3 (e.g. query_test/test_udfs.py and query_test/test_scanners.py) Change-Id: If8b3020826a2f376815016affc7fd4c8634b3cba Reviewed-on: http://gerrit.cloudera.org:8080/23083 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Riza Suminto <[email protected]> Reviewed-by: Csaba Ringhofer <[email protected]> --- M tests/common/impala_connection.py 1 file changed, 7 insertions(+), 0 deletions(-) Approvals: Impala Public Jenkins: Verified Riza Suminto: Looks good to me, approved Csaba Ringhofer: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/23083 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If8b3020826a2f376815016affc7fd4c8634b3cba Gerrit-Change-Number: 23083 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
