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

Allow mixed case partitions for Hive sync_partition_metadata #3431

Closed
electrum opened this issue Apr 14, 2020 · 1 comment · Fixed by #3830
Closed

Allow mixed case partitions for Hive sync_partition_metadata #3431

electrum opened this issue Apr 14, 2020 · 1 comment · Fixed by #3830
Assignees
@kondamudikarthik
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone
335

Comments

@electrum
Copy link
Member

electrum commented Apr 14, 2020 •
edited by findepi
Loading

The Hive system.sync_partition_metadata procedure allows discovering partitions on the file system that match the Hive naming convention. For compatibility with Hive's MSCK REPAIR TABLE, partition names must be in lowercase by default. However, for users that wish to use it with mixed case names, we can add a new case_insensitive argument to allow this behavior.
@electrum electrum added enhancement New feature or request good first issue Good for newcomers labels Apr 14, 2020
@kondamudikarthik
Copy link

Hello, I would like to work on this. Thanks!

@aalbu aalbu mentioned this issue May 23, 2020
Merged
@findepi findepi added this to the 335 milestone Jun 1, 2020
@findepi findepi mentioned this issue Jun 1, 2020
Closed
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kondamudikarthik kondamudikarthik

Labels
enhancement New feature or request good first issue Good for newcomers
Milestone
335
Development

Successfully merging a pull request may close this issue.

Handle mixed case partition names in sync_partition_metadata aalbu/trino
3 participants
@electrum @findepi @kondamudikarthik