Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-06-01 Thread via GitHub
nastra commented on code in PR #13130: URL: https://github.com/apache/iceberg/pull/13130#discussion_r2120172752 ## hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java: ## @@ -1209,11 +1208,4 @@ public void testDatabaseLocationWithSlashInWarehouseDir() {

Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-30 Thread via GitHub
jmelinav commented on code in PR #13130: URL: https://github.com/apache/iceberg/pull/13130#discussion_r2115914352 ## hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java: ## @@ -1209,11 +1208,4 @@ public void testDatabaseLocationWithSlashInWarehouseDir() {

Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-30 Thread via GitHub
jmelinav commented on code in PR #13130: URL: https://github.com/apache/iceberg/pull/13130#discussion_r2115914352 ## hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java: ## @@ -1209,11 +1208,4 @@ public void testDatabaseLocationWithSlashInWarehouseDir() {

Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-28 Thread via GitHub
pvary commented on code in PR #13130: URL: https://github.com/apache/iceberg/pull/13130#discussion_r2110933234 ## hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java: ## @@ -1209,11 +1208,4 @@ public void testDatabaseLocationWithSlashInWarehouseDir() {

Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-27 Thread via GitHub
pvary commented on code in PR #13130: URL: https://github.com/apache/iceberg/pull/13130#discussion_r2110933234 ## hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java: ## @@ -1209,11 +1208,4 @@ public void testDatabaseLocationWithSlashInWarehouseDir() {

Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-27 Thread via GitHub
jmelinav commented on code in PR #13130: URL: https://github.com/apache/iceberg/pull/13130#discussion_r2110153869 ## hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java: ## @@ -1209,11 +1208,4 @@ public void testDatabaseLocationWithSlashInWarehouseDir() {

Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-27 Thread via GitHub
pvary commented on code in PR #13130: URL: https://github.com/apache/iceberg/pull/13130#discussion_r2110052098 ## hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java: ## @@ -1209,11 +1208,4 @@ public void testDatabaseLocationWithSlashInWarehouseDir() {

Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-27 Thread via GitHub
jmelinav commented on code in PR #13130: URL: https://github.com/apache/iceberg/pull/13130#discussion_r2109851143 ## hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java: ## @@ -1209,11 +1208,4 @@ public void testDatabaseLocationWithSlashInWarehouseDir() {

Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-26 Thread via GitHub
pvary commented on code in PR #13130: URL: https://github.com/apache/iceberg/pull/13130#discussion_r2106791809 ## hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java: ## @@ -1209,11 +1208,4 @@ public void testDatabaseLocationWithSlashInWarehouseDir() {

Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-23 Thread via GitHub
nastra commented on code in PR #13130: URL: https://github.com/apache/iceberg/pull/13130#discussion_r2104562932 ## hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java: ## @@ -1212,7 +1212,6 @@ public void testDatabaseLocationWithSlashInWarehouseDir() {

Re: [PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-23 Thread via GitHub
pvary commented on PR #13130: URL: https://github.com/apache/iceberg/pull/13130#issuecomment-2903722488 @jmelinav: Could you please run `./gradlew spotlessApply` and `./gradlew checkstyleMain`? -- This is an automated message from the Apache Git Service. To respond to the message, please

[PR] Hive: Throw exception for when listing a non-existing namespace [iceberg]

2025-05-22 Thread via GitHub
jmelinav opened a new pull request, #13130: URL: https://github.com/apache/iceberg/pull/13130 Throw exception for when listing a non-existing namespace Fixes https://github.com/apache/iceberg/issues/12874 -- This is an automated message from the Apache Git Service. To respond to the