[GitHub] [incubator-iceberg] rdblue commented on issue #509: Remove flaky Hive table tests

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

rdblue commented on issue #509: Remove flaky Hive table tests
URL: https://github.com/apache/incubator-iceberg/pull/509#issuecomment-537106532
 
 
   > do we think CATALOG_CACHE in HiveCatalogs is related to the problem with these tests?
   
   Yes! I forgot about that one, but it puts us over the limit: Spark will have its own connection from the built-in metastore. Then we have a pool from `HiveCatalogs` that is reused, with two connections. One more connection for the direct client used to clean up, and two more for the `HiveCatalog` instance. That comes to 6, which is over the limit. Maybe if we add back the change to share the client pool for cleanup with the catalog used in tests?
   
   Let's bring these tests back in a PR, but keep this in master for now to unblock the release and avoid so many test failures for PRs.

----------------------------------------------------------------
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


With regards,
Apache Git Services

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