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.

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

[#461][MINOR] docs(hive): add doc and IT for Hive column position change #496

Merged
merged 4 commits into from
Oct 12, 2023

Conversation

mchades
Copy link
Contributor

@mchades mchades commented Oct 11, 2023

What changes were proposed in this pull request?

add doc and IT for Hive column position change

Why are the changes needed?

Clearer usage of altering Hive column position

Fix: #461

Does this PR introduce any user-facing change?

no

How was this patch tested?

IT added

@github-actions
Copy link

github-actions bot commented Oct 11, 2023

Code Coverage Report

Overall Project 62.41% -0.05% 🟢
Files changed 0% 🔴

Module Coverage
catalog-lakehouse-iceberg 79% 🟢
catalog-hive 64.36% -0.45% 🔴
Files
Module File Coverage
catalog-lakehouse-iceberg IcebergCatalogOperations.java 76.53% 🟢
catalog-hive HiveCatalogOperations.java 67.34% -1.42% 🔴

@jerryshao
Copy link
Contributor

Is it possible that we describe the unexpected behavior clearly in exception message, so that user will easily know it is the Hive's behavior?

@jerryshao jerryshao merged commit 7d87fd3 into apache:main Oct 12, 2023
2 checks passed
Clearvive pushed a commit that referenced this pull request Oct 17, 2023
…nge (#496)

### What changes were proposed in this pull request?
add doc and IT for Hive column position change


### Why are the changes needed?
Clearer usage of altering Hive column position

Fix: #461 

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
IT added
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug report] updateColumnPosition doesn't work as expected
2 participants