PDGGK commented on PR #17492:
URL: https://github.com/apache/iceberg/pull/17492#issuecomment-5248529637
@uros-b friendly ping — your checkstyle point was the only review comment
and it's been resolved since `85c9952`: the assertion now chains
`.hasMessage("Interrupted in SQL command")`, matching `listNamespaces` (:203)
and `listViews` (:305) in the same file. I verified the rule actually fires by
removing the message check again and watching
`AssertThatThrownByWithMessageCheck` fail, rather than just assuming the build
was happy.
All 36 checks are green. The change itself is one line —
`JdbcCatalog.execute` was the only one of the four `InterruptedException`
handlers in the class not restoring the interrupt status, while
`initializeCatalogTables`, `updateSchemaIfRequired` and `fetch` all do.
No rush if you'd rather someone else take it — just flagging that it's ready.
--
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]
