Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23509 )

Change subject: KUDU-3478 Add Python client examples test script
......................................................................


Patch Set 2: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23509/1/python/Makefile
File python/Makefile:

http://gerrit.cloudera.org:8080/#/c/23509/1/python/Makefile@25
PS1, Line 25: ifdef TEST
            :   python -m unittest $(TEST)
            : else
            :   python setup.py test
            : endif
> Personally I use `make test` in the Python folder to quickly run all the te
OK, I guess it makes sense to keep different set of tests under different 
scopes.

If not including the run of test-python-examples.sh under the existing 'test' 
target, does it make sense at least to add a separate target to run it instead 
of running it 'by hand'?  At least, it would be a handy shortcut to run all the 
existing Python example scripts instead of building a command line for a manual 
run of each of them.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic2ea271a8d33a285e4bf222afa17ec77583a5b8b
Gerrit-Change-Number: 23509
Gerrit-PatchSet: 2
Gerrit-Owner: Marton Greber <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
Gerrit-Comment-Date: Thu, 09 Oct 2025 20:12:44 +0000
Gerrit-HasComments: Yes

Reply via email to