aokolnychyi commented on code in PR #11087: URL: https://github.com/apache/iceberg/pull/11087#discussion_r1746503833
########## site/docs/contribute.md: ########## @@ -388,6 +388,34 @@ When passing boolean arguments to existing or external methods, use inline comme dropTable(identifier, purge); ``` +#### Accessing instance variables Review Comment: I wanted to look into ways to automate this after documenting. I took a look now. It doesn't seem there is an existing checkstyle rule that would enforce the behavior we are looking for. We may try doing something with regular expressions but it may be hard to capture the context. -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
