amogh-jahagirdar commented on code in PR #13386:
URL: https://github.com/apache/iceberg/pull/13386#discussion_r2167147736
##########
core/src/main/java/org/apache/iceberg/rest/ParserContext.java:
##########
@@ -44,7 +44,7 @@ static Builder builder() {
}
static class Builder {
- private Map<String, Object> data;
+ private final Map<String, Object> data;
Review Comment:
Yeah good eye, I think we would've caught this when implementing the actual
client side changes for scan planning w tests, we wouldn't have any of the
expected context passed through but it's good to catch this now.
--
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
