kbendick opened a new pull request #1580: URL: https://github.com/apache/iceberg/pull/1580
In many places we rely on `table.toString()` when we're really looking for the table's name, which is available in another method `table.name()`. Although at the moment, at least when looking at the interface, the two are semantically equivalent, it makes it much more difficult to read the code when seeing so many calls to `table.toString()` when what we're really looking for is the table's name. This assists with, but does not necessarily close, issue https://github.com/apache/iceberg/issues/1544. I'm not sure if this has captured all instances where `table.toString` is used when logically `table.name` is what should be requested. ---------------------------------------------------------------- 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]
