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 2022/02/09 17:28:08 UTC

[GitHub] [iceberg] amogh-jahagirdar edited a comment on pull request #4076: Nessie: minor cleanup. Consistent naming in LOG and fixing typo in UpdateableClassReference

amogh-jahagirdar edited a comment on pull request #4076:
URL: https://github.com/apache/iceberg/pull/4076#issuecomment-1034012785


   > the LOG renaming changes LGTM, but I believe both spellings are correct/common. Either way I don't have any strong opinions on either spelling, so I'll let somebody else decide :)
   
   Sounds good! Yeah just something I noticed, I don't have a strong preference either, more so that it is consistent.
   Did not know this about Updatable vs Updateable :) so I am also okay with either, whatever the community prefers. Thanks for the quick look! 


-- 
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