Skip to content

Convert TableMetadata to record#21594

Merged
wendigo merged 2 commits intomasterfrom
serafin/metadata-records-v4
Apr 17, 2024
Merged

Convert TableMetadata to record#21594
wendigo merged 2 commits intomasterfrom
serafin/metadata-records-v4

Conversation

@wendigo
Copy link
Copy Markdown
Contributor

@wendigo wendigo commented Apr 17, 2024

Extracted from #21572

@cla-bot cla-bot bot added the cla-signed label Apr 17, 2024
@wendigo wendigo requested a review from findepi April 17, 2024 14:05
@github-actions github-actions bot added tests:hive hive Hive connector labels Apr 17, 2024
@findepi findepi requested a review from martint April 17, 2024 15:59
@findepi
Copy link
Copy Markdown
Member

findepi commented Apr 17, 2024

should TableSchema converstion be here too?
the classes should have ~same API (generally one is simpler version of the other)

@wendigo
Copy link
Copy Markdown
Contributor Author

wendigo commented Apr 17, 2024

@findepi added here

@wendigo wendigo merged commit 1543369 into master Apr 17, 2024
@wendigo wendigo deleted the serafin/metadata-records-v4 branch April 17, 2024 18:46
@github-actions github-actions bot added this to the 445 milestone Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@martint martint martint approved these changes

@findepi findepi Awaiting requested review from findepi

Assignees

No one assigned

Labels

cla-signed hive Hive connector

Milestone

445

Development

Successfully merging this pull request may close these issues.

3 participants

@wendigo @findepi @martint