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

HIVE-27057 : Revert "HIVE-21741 Backport HIVE-20221 & related fix HIVE-20833 to branch-3" #4156

Closed

Conversation

amanraj2520
Copy link
Contributor

@amanraj2520 amanraj2520 commented Mar 27, 2023

JIRA link : https://issues.apache.org/jira/browse/HIVE-27057

This reverts commit e3d5abd

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

…anch-3: Increase column width for partition_params (David Lavati via Alan Gates)"

This reverts commit e3d5abd
@amanraj2520
Copy link
Contributor Author

@vihangk1 As suggested by you, this revert fixed the BASIC_STATS printed in a json string issue. But there is another failure in the sysdb.q file which is why I have raised #4157. First we should merge this PR (#4157) and revert this. I have tested in my local. It is working fine. Can you please approve and merge the #4157 PR.

@amanraj2520 amanraj2520 changed the title Revert "HIVE-21741 Backport HIVE-20221 & related fix HIVE-20833 to branch-3" HIVE-27057 : Revert "HIVE-21741 Backport HIVE-20221 & related fix HIVE-20833 to branch-3" Mar 27, 2023
@amanraj2520
Copy link
Contributor Author

@vihangk1 Can you please approve this

@vihangk1
Copy link
Contributor

I merged the PR which disables sysdb.q on branch-3. Lets create a ticket to investigate the failure separately. I don't think we should revert this PR right away because this is an important fix. We can revert these commits if we determine that this issue is indeed a 3.2 blocker.

@amanraj2520
Copy link
Contributor Author

I agree that this ticket has important fixes that should not be reverted. Closing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants