Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/21816 )
Change subject: KUDU-3604: Pin Cython Version, Package Maintenance ...................................................................... Patch Set 3: Ideally if a python package is installed the expected behaviour is that dependencies are pulled during the install process automatically. With this patch the Kudu Python package follows this workflow whereas the previous way was the non-conventional. >From the README I removed the line which instructed to run pip install cython >before running pip install kudu-python. I figured that this way we dont need any additional docs. Maybe in the release notes we can mention this. I hope I answered your question Abhishek. -- To view, visit http://gerrit.cloudera.org:8080/21816 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I45ad45c56fc3f3ecf57a413edbd0866fccbd8f79 Gerrit-Change-Number: 21816 Gerrit-PatchSet: 3 Gerrit-Owner: Marton Greber <greber...@gmail.com> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Ashwani Raina <ara...@cloudera.com> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <greber...@gmail.com> Gerrit-Reviewer: Zoltan Chovan <zcho...@cloudera.com> Gerrit-Comment-Date: Tue, 01 Oct 2024 15:41:44 +0000 Gerrit-HasComments: No