rdblue commented on pull request #1640: URL: https://github.com/apache/iceberg/pull/1640#issuecomment-721423980
What's the drawback of using a map? It is a bit narrower to use the function, but it is also limiting. Catalogs can't use `getOrDefault` or check `contains`. I also don't have a serious problem if we need to copy to create an Immutable map for this because it isn't in a performance-critical path. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
