Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-11-16 Thread via GitHub
github-actions[bot] commented on PR #10845: URL: https://github.com/apache/iceberg/pull/10845#issuecomment-2480862705 This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If

Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-11-16 Thread via GitHub
github-actions[bot] closed pull request #10845: Hive: Pre-check namespace presence for listTables URL: https://github.com/apache/iceberg/pull/10845 -- 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 th

Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-11-09 Thread via GitHub
github-actions[bot] commented on PR #10845: URL: https://github.com/apache/iceberg/pull/10845#issuecomment-2466519789 This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pul

Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-08-12 Thread via GitHub
pvary commented on PR #10845: URL: https://github.com/apache/iceberg/pull/10845#issuecomment-2283750870 So basically we don't get the expected results from the listing of the tables, so we issue another HMS call (and accept the performance degradation), so we can return the correct answer o

Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-08-12 Thread via GitHub
pvary commented on code in PR #10845: URL: https://github.com/apache/iceberg/pull/10845#discussion_r1713603097 ## hive-metastore/src/test/java/org/apache/iceberg/hive/HiveTableTest.java: ## @@ -351,6 +352,11 @@ public void testListTables() throws TException, IOException {

Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-08-07 Thread via GitHub
deniskuzZ commented on PR #10845: URL: https://github.com/apache/iceberg/pull/10845#issuecomment-2272810150 Hive is throwing an exception when trying to list tables in non-existing catalog: show tables in UNKNOWN; INFO : Compiling command(queryId=hive_20240807072922_373feca6-

Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-08-06 Thread via GitHub
haizhou-zhao commented on PR #10845: URL: https://github.com/apache/iceberg/pull/10845#issuecomment-2272383395 @pvary I downloaded Hive 4.0.0 binary and did a quick test. I believe listing tables on a DB that does not existing in 4.0.0 will also return an empty list, not throwing `UnknownDB

Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-08-01 Thread via GitHub
pvary commented on PR #10845: URL: https://github.com/apache/iceberg/pull/10845#issuecomment-2263912533 Maybe @deniskuzZ could help with info about Hive 4.0.0 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-08-01 Thread via GitHub
haizhou-zhao commented on PR #10845: URL: https://github.com/apache/iceberg/pull/10845#issuecomment-2263890194 @pvary I know hive 2.3.9 and 3.1.2 returns empty list, which is close but a bit different from what you are asking for. Let me know if you are looking for 2.3.10 and 3.1.3

Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-08-01 Thread via GitHub
pvary closed pull request #10845: Hive: Pre-check namespace presence for listTables URL: https://github.com/apache/iceberg/pull/10845 -- 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 com

Re: [PR] Hive: Pre-check namespace presence for listTables [iceberg]

2024-08-01 Thread via GitHub
pvary commented on PR #10845: URL: https://github.com/apache/iceberg/pull/10845#issuecomment-2263842502 @haizhou-zhao: which versions of HMS return empty list, and which version return `UnknownDBException`? I'm specifically interested in the supported HMS versions (2. -- This is an aut