Re: [PR] [JDBC] Part3: Plumb JDBC module to quarkus [polaris]

[via GitHub]

singhpk234 commented on code in PR #1371:
URL: https://github.com/apache/polaris/pull/1371#discussion_r2059026442


##########
quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/IcebergCatalogTest.java:
##########
@@ -1846,4 +1847,28 @@ public void 
testConcurrencyConflictUpdateTableDuringFinalTransaction() {
   private static InMemoryFileIO getInMemoryIo(IcebergCatalog catalog) {
     return (InMemoryFileIO) ((ExceptionMappingFileIO) 
catalog.getIo()).getInnerIo();
   }
+
+  @Test
+  @Disabled
+  @Override
+  public void createAndDropEmptyNamespace() {
+    // TODO: remove this when quarkus bug : 
https://github.com/quarkusio/quarkus/issues/46642

Review Comment:
   @snazy  i tried this unfortunately this is not working or i might be missing 
something 
   ![Screenshot 2025-04-24 at 10 55 24 
AM](https://github.com/user-attachments/assets/4080fb77-742a-4df5-a63f-2725374baa3e)
   
   Though if we have 
   
   ```
     static {
       
Assumptions.setPreferredAssumptionException(PreferredAssumptionException.JUNIT5);
     }
   ```
   
   my read is they are doing same thing, let me debug this more, removed this 
for meanwhile 



-- 
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: issues-unsubscr...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org