[GitHub] [iceberg] rdblue commented on issue #1513: inconsistent handling when TableOperations#current() returns null

[GitBox <gi...@apache.org>]

rdblue commented on issue #1513:
URL: https://github.com/apache/iceberg/issues/1513#issuecomment-700205519


   `current` will return `null` only after refreshing the table throws `NoSuchTableException`. I think this is minor because callers would have ignored that exception to fail when current returns null.
   
   That said, I think we can throw the exception and set `shouldRefresh = true`. That way, any call to get table metadata will trigger a refresh and either get the table back to a consistent state, or fail with the same exception.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org