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-24788: Backport HIVE-23338 to branch-3.1 #1986

Merged
merged 1 commit into from
Feb 24, 2021

Conversation

h-vetinari
Copy link

Trying to backport 3193589 for HIVE-24788

@h-vetinari h-vetinari changed the title [WIP] HIVE-24788: Backport HIVE-23338 to branch-3.1 HIVE-24788: Backport HIVE-23338 to branch-3.1 Feb 16, 2021
@h-vetinari
Copy link
Author

Tests seem to pass, removing WIP-label

@h-vetinari
Copy link
Author

This is my first time contributing to hive. I was surprised that another CI run appeared 24h later. It fails, but says

There are 0 new tests failing, 26 existing failing and 53 skipped.

That sounds like the failures are pre-existing? Anything I need to do here?

@h-vetinari
Copy link
Author

Hello @jcamachor @pvargacl, I'm unfamiliar with the process here, would you be able to have a look at this PR (or tell me what's missing), please? 🙃

@pvargacl
Copy link
Contributor

@h-vetinari The precommit tests are unfortunately flaky on the branch 3.1, so do not expect a green run there.

@klcopp can you take a look

@klcopp
Copy link
Contributor

klcopp commented Feb 23, 2021

All 26 test failures were seen previously on http://ci.hive.apache.org/job/hive-precommit/job/PR-1952/1/#showFailuresLink, so don't worry about those failures. +1 and I can commit in 24h.

@klcopp klcopp self-requested a review February 23, 2021 11:29
@klcopp klcopp merged commit 0361dd9 into apache:branch-3.1 Feb 24, 2021
udaynpusa pushed a commit to mapr/hive that referenced this pull request Jan 30, 2024
…by Karen Coppage)

HIVE-23338: Bump jackson version to 2.10.0
Closes apache#1986.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants