Skip to content

[AMORO-4350][catalog] Fix Hive view handling in table management - #4351

Open
csurong wants to merge 2 commits into
apache:masterfrom
csurong:fix/hive-view-filtering
Open

[AMORO-4350][catalog] Fix Hive view handling in table management#4351
csurong wants to merge 2 commits into
apache:masterfrom
csurong:fix/hive-view-filtering

Conversation

@csurong

@csurong csurong commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Why are the changes needed?

Hive views may be returned by Hive Metastore listing APIs and exposed as manageable tables. Opening one can make MixedHiveCatalog dereference a null storage location and return HTTP 500.

This PR excludes Hive virtual and materialized views from Amoro table management paths and rejects them before physical table metadata is accessed.

Closes #4350.

Brief change log

  • Add lightweight HMS table metadata access and centralized Hive view classification.
  • Filter views from native Hive, Iceberg HiveCatalog, and Mixed Hive listings.
  • Reject views and invalid physical metadata during table loading.
  • Add unit and integration coverage for Hive, Iceberg, and Mixed Hive paths.

How was this patch tested?

  • Add test cases that check the changes thoroughly including negative and positive cases if possible.
  • Add screenshots for manual tests if appropriate.
  • Run tests locally before making a pull request.

The targeted unit and integration tests passed. AMS compilation, Hadoop 2/Hive 2.3 compilation, Spotless, Checkstyle, and diff checks also passed.

Documentation

  • Does this pull request introduce a new feature? no
  • If yes, how is the feature documented? not applicable

@github-actions github-actions Bot added module:ams-server Ams server module module:mixed-hive Hive moduel for Mixed Format module:common labels Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:ams-server Ams server module module:common module:mixed-hive Hive moduel for Mixed Format

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Opening a Hive view from the Tables page causes a NullPointerException

1 participant