Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Core: Add metadataFileLocation in TableUtil #12082

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Core: Add metadataFileLocation in TableUtil #12082

wants to merge 2 commits into from

Conversation

dramaticlly
Copy link
Contributor

Context: #11931 (comment)

  • Add a new static helper method in TableUtil to expose metadata file location for a given table
  • also added a public method in serializableTable to expose metadata file location
  • leverage EnumSource in unit tests to help with execute in oarameterized fashion for all metadata tables

@amogh-jahagirdar @nastra if you want to take a look

@nastra
Copy link
Contributor

nastra commented Jan 24, 2025

I'll wait a bit with merging in case @amogh-jahagirdar has any comments

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ebyhr ebyhr ebyhr approved these changes

@nastra nastra nastra approved these changes

Assignees
No one assigned
Labels
core spark
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dramaticlly @nastra @ebyhr