You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2021/08/02 11:12:43 UTC

[GitHub] [iceberg] TGooch44 commented on pull request #2910: [python] add additional projection util methods

TGooch44 commented on pull request #2910:
URL: https://github.com/apache/iceberg/pull/2910#issuecomment-890940364


   > imho classes full of static methods are very weird in python. This module should just contain functions
   
   I'll take the blame for setting this precedent here.  I agree with @rymurr though,  these are better off as just plain functions.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org