kevinjqliu commented on issue #13366: URL: https://github.com/apache/iceberg/issues/13366#issuecomment-2996823048
Some extra context: Looks like the JDBC catalog has unit tests specifically for concurrency. See `testConcurrentFastAppends` and `testConcurrentConnections` in [`TestJdbcTableConcurrency.java`](https://github.com/apache/iceberg/blob/6e432fcb24fd55c9024a3192b17a364623886047/core/src/test/java/org/apache/iceberg/jdbc/TestJdbcTableConcurrency.java) Whats interesting is that the tests all use file-based sqlite connection string, not in-memory ones https://github.com/apache/iceberg/blob/6e432fcb24fd55c9024a3192b17a364623886047/core/src/test/java/org/apache/iceberg/jdbc/TestJdbcTableConcurrency.java#L116 This makes sense since the "original" REST docker image (`tabulario/iceberg-rest`) actually used a file-based sqlite lite connection string https://github.com/databricks/iceberg-rest-image/blob/62a5078032e0675d34b377d2ead2eea814c1da48/Dockerfile#L33 And this was changed when we created the new `iceberg-rest-fixture` https://github.com/apache/iceberg/pull/11283/files#diff-b16b276a077c08a7178227bc5f8cca28765c13e4e1563cf87f15835e9ade2833R34 So i'd classify this as a "regression" -- 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...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
