You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Jordan Birdsell (Code Review)" <ge...@cloudera.org> on 2018/06/18 16:31:07 UTC

[kudu-CR] python-client: fix test scan rows string predicate and projection

Jordan Birdsell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10255 )

Change subject: python-client: fix test_scan_rows_string_predicate_and_projection
......................................................................

python-client: fix test_scan_rows_string_predicate_and_projection

Revert unneccesary change to test in test_scanner.py in commit
3bf125f469a2929f7fe20e4e31f073376f37575d. The test
test_scan_rows_string_predicate_and_projection was already working
and using column names is both easier to read and more reliable in
the even of a schema change than the column index numbers.

Change-Id: I12c900c1366fad61f5c04d023d80fea1f488ef19
Reviewed-on: http://gerrit.cloudera.org:8080/10255
Tested-by: Kudu Jenkins
Reviewed-by: David Ribeiro Alves <da...@gmail.com>
---
M python/kudu/tests/test_scanner.py
1 file changed, 1 insertion(+), 3 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  David Ribeiro Alves: Looks good to me, approved

-- 
To view, visit http://gerrit.cloudera.org:8080/10255
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I12c900c1366fad61f5c04d023d80fea1f488ef19
Gerrit-Change-Number: 10255
Gerrit-PatchSet: 6
Gerrit-Owner: andy@phdata.io
Gerrit-Reviewer: David Ribeiro Alves <da...@gmail.com>
Gerrit-Reviewer: Jordan Birdsell <jt...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: andy@phdata.io