Skip to content

PHOENIX-7613 Update Apache parent pom to 31 #2172

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

PHOENIX-7613 Update Apache parent pom to 31 #2172

wants to merge 1 commit into from
+40 −13

Conversation

stoty
Copy link
Contributor

@stoty stoty commented Jun 2, 2025

No description provided.

stoty
stoty commented Aug 4, 2025
View reviewed changes
phoenix-core-client/src/main/java/org/apache/phoenix/schema/MetaDataClient.java Outdated
/**
* To check if TTL is defined at any of the child below we are checking it at
* {@link org.apache.phoenix.coprocessor.MetaDataEndpointImpl#mutateColumn(List, ColumnMutator, int, PTable, PTable, boolean)}
* level where in function
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is fixing an unrelated spotless violation that crept in.

@stoty
Copy link
Contributor Author

stoty commented Aug 5, 2025

This causes new dependcy errors to be reported.
Gonna have to work on this some more.

@stoty stoty marked this pull request as draft August 5, 2025 05:00
@stoty stoty requested review from NihalJain and richardantal August 5, 2025 08:19
@stoty
Copy link
Contributor Author

stoty commented Aug 5, 2025 •
edited
Loading

I had to clean up some dependencies to satisfy the new maven-dependency-plugin.
Please take another look @NihalJain @richardantal

@stoty stoty marked this pull request as ready for review August 5, 2025 08:20
@stoty
Copy link
Contributor Author

stoty commented Aug 6, 2025

Thanks.
Gonna hold this until until I fix 5.1 with Hbase 2.5.

@stoty
PHOENIX-7613 Update Apache parent pom to 31
60b3653 
also clean up some test/compile dependency scopes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NihalJain NihalJain NihalJain approved these changes

@richardantal richardantal Awaiting requested review from richardantal

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stoty @NihalJain @richardantal