Skip to content

Move ensure in boundaries of project lock #2470

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
Merged
merged 1 commit into from
Jun 12, 2025
Merged

Move ensure in boundaries of project lock #2470

merged 1 commit into from
Jun 12, 2025

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Jun 11, 2025

WIP

@cstamas cstamas self-assigned this Jun 11, 2025
@cstamas cstamas added the mvn3 label Jun 11, 2025
gnodet
gnodet approved these changes Jun 11, 2025
View reviewed changes
Copy link
Contributor

@gnodet gnodet left a comment

Choose a reason for hiding this comment

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

LGTM. This will have to be ported to Maven 4.
Can we double check if the original issue is fixed by this PR ?

@cstamas cstamas mentioned this pull request Jun 11, 2025
Closed
@cstamas
Copy link
Member Author

cstamas commented Jun 11, 2025

Seems the issue is fixed with this PR as well.

@cstamas cstamas marked this pull request as ready for review June 11, 2025 21:42
@cstamas cstamas merged commit 69267a1 into apache:maven-3.9.x Jun 12, 2025
18 checks passed
@cstamas cstamas deleted the maven-3.9.x-move-into-lock branch June 12, 2025 10:36
@github-actions github-actions bot added this to the 3.9.11 milestone Jun 12, 2025
@slawekjaranowski slawekjaranowski added the bug Something isn't working label Jun 18, 2025
@dongjoon-hyun dongjoon-hyun mentioned this pull request Jul 15, 2025
Closed
dongjoon-hyun added a commit to apache/orc that referenced this pull request Jul 15, 2025
@dongjoon-hyun
ORC-1958: Upgrade Maven to 3.9.11
ab9d7d0 
### What changes were proposed in this pull request?

This PR aims to upgrade Maven to 3.9.11.

### Why are the changes needed?

To bring the latest bug fixes.
- https://github.com/apache/maven/releases/tag/maven-3.9.11
  - apache/maven#2489
  - apache/maven#2470
  - apache/maven#10896
  - apache/maven#2574

### How was this patch tested?

Pass the CIs.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #2331 from dongjoon-hyun/ORC-1958.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
dongjoon-hyun added a commit to apache/orc that referenced this pull request Jul 15, 2025
@dongjoon-hyun
ORC-1958: Upgrade Maven to 3.9.11
6effd1d 
### What changes were proposed in this pull request?

This PR aims to upgrade Maven to 3.9.11.

### Why are the changes needed?

To bring the latest bug fixes.
- https://github.com/apache/maven/releases/tag/maven-3.9.11
  - apache/maven#2489
  - apache/maven#2470
  - apache/maven#10896
  - apache/maven#2574

### How was this patch tested?

Pass the CIs.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #2331 from dongjoon-hyun/ORC-1958.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
(cherry picked from commit ab9d7d0)
Signed-off-by: Dongjoon Hyun <[email protected]>
@LuciferYang LuciferYang mentioned this pull request Jul 17, 2025
Closed
dongjoon-hyun pushed a commit to apache/spark that referenced this pull request Jul 17, 2025
@LuciferYang @dongjoon-hyun
[SPARK-52845][BUILD] Upgrade maven to 3.9.11
7a93108 
### What changes were proposed in this pull request?
This pr aims to upgrade maven 3.9.10 to 3.9.11.

### Why are the changes needed?
The new version brings some bug fixes:

- apache/maven#2489
- apache/maven#2470

The full release notes as follows:

- https://github.com/apache/maven/releases/tag/maven-3.9.11

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

### How was this patch tested?
- Pass Github Actions

### Was this patch authored or co-authored using generative AI tooling?
No

Closes #51491 from LuciferYang/maven-3.9.11.

Lead-authored-by: yangjie01 <[email protected]>
Co-authored-by: YangJie <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
haoyangeng-db pushed a commit to haoyangeng-db/apache-spark that referenced this pull request Jul 22, 2025
@LuciferYang @haoyangeng-db
[SPARK-52845][BUILD] Upgrade maven to 3.9.11
5a3cadc 
### What changes were proposed in this pull request?
This pr aims to upgrade maven 3.9.10 to 3.9.11.

### Why are the changes needed?
The new version brings some bug fixes:

- apache/maven#2489
- apache/maven#2470

The full release notes as follows:

- https://github.com/apache/maven/releases/tag/maven-3.9.11

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

### How was this patch tested?
- Pass Github Actions

### Was this patch authored or co-authored using generative AI tooling?
No

Closes apache#51491 from LuciferYang/maven-3.9.11.

Lead-authored-by: yangjie01 <[email protected]>
Co-authored-by: YangJie <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gnodet gnodet gnodet approved these changes

@slawekjaranowski slawekjaranowski slawekjaranowski approved these changes

Assignees

@cstamas cstamas

Labels
bug Something isn't working mvn3
Projects
None yet
Milestone
3.9.11
Development

Successfully merging this pull request may close these issues.
3 participants
@cstamas @gnodet @slawekjaranowski