mengw15 commented on PR #7539: URL: https://github.com/apache/texera/pull/7539#issuecomment-5284755864
On coverage: raised in 22afc3e6b with cases for the close-failure path (a catalog whose `close()` throws is still removed and nothing escapes), a catalog that is not `AutoCloseable` (hadoop/postgres need not be), and the `ExecutionError` unwrap. The one path that stays uncovered is the cache-miss loader actually building a catalog — that would need a live REST server, so covering it would make the unit suite depend on external infrastructure. -- 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]
