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

PHOENIX-6981 Update Jackson to 2.14.1 #1614

Merged
merged 1 commit into from
Jun 16, 2023

Conversation

ksobolew
Copy link
Contributor

If that's too much of a jump, then we could bump instead to 2.12.7.1 :)

pom.xml Outdated Show resolved Hide resolved
@gjacoby126
Copy link
Contributor

@ksobolew - change looks good. Could you please change the PR and the commit message to include the JIRA number?

@ksobolew
Copy link
Contributor Author

@ksobolew - change looks good. Could you please change the PR and the commit message to include the JIRA number?

@gjacoby126 I don't have a JIRA number. How do I get one?

@gjacoby126
Copy link
Contributor

@ksobolew - our issue tracking system is https://issues.apache.org/jira/ . Unfortunately, public signup for it was recently turned off because of bot problems, but if you request an account at https://selfserve.apache.org/jira-account.html I or one of the other PMC members can approve you and make you a contributor to the project. (This step's a one-time thing.)

Then it's just a matter of filing a JIRA ticket and rebasing the PR so that the commit message includes the JIRA number.

@ksobolew ksobolew changed the title Update Jackson to 2.15.1 Update Jackson to 2.14.1 Jun 16, 2023
@ksobolew ksobolew changed the title Update Jackson to 2.14.1 PHOENIX-6981 Update Jackson to 2.14.1 Jun 16, 2023
@ksobolew
Copy link
Contributor Author

@gjacoby126 Done!

@ksobolew
Copy link
Contributor Author

(I requested a new account, but then realized that I already have one, so I used that one instead.)

Copy link
Contributor

@gjacoby126 gjacoby126 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, thanks @ksobolew

@gjacoby126 gjacoby126 merged commit ce52f50 into apache:master Jun 16, 2023
@ksobolew ksobolew deleted the kudi/bump-jackson branch June 16, 2023 13:26
gjacoby126 pushed a commit that referenced this pull request Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants