kbendick commented on code in PR #5118:
URL: https://github.com/apache/iceberg/pull/5118#discussion_r907823638
##########
core/src/main/java/org/apache/iceberg/rest/responses/LoadTableResponse.java:
##########
@@ -94,6 +96,11 @@ public Builder withTableMetadata(TableMetadata
tableMetadata) {
return this;
}
+ public Builder withMetadataLocation(String metaLocation) {
+ this.metadataLocation = metaLocation;
+ return this;
+ }
Review Comment:
This isn't used presently, but I've found myself having to rebuild the
`TableMetadata` in a few places to get `metadataLocation` to be non-null.
Given that the accessor in `LoadTableResponse` _also_ calls the builder and
adds `metadataLocation` to it, this could be useful.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
