Skip to content

Conversation

@vibhatha
Copy link
Contributor

Rationale for this change

This PR was created to replace #39374 and do the necessary changes for org.codehaus.mojo upgrade to take place.

What changes are included in this PR?

The changes to the org.codehaus.mojo version and an upgrade on the maven version used in the .env.

Are these changes tested?

Tested locally, but this requires a CI verification on Java.

Are there any user-facing changes?

No

@vibhatha
Copy link
Contributor Author

@github-actions crossbow submit java

@github-actions
Copy link

Revision: e1c246622418456d6582df60db993b4a67b0166c

Submitted crossbow builds: ursacomputing/crossbow @ actions-723588355a

Task Status
java-jars GitHub Actions
verify-rc-source-java-linux-almalinux-8-amd64 GitHub Actions
verify-rc-source-java-linux-conda-latest-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-20.04-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-22.04-amd64 GitHub Actions
verify-rc-source-java-macos-amd64 GitHub Actions

@vibhatha vibhatha marked this pull request as ready for review January 19, 2024 03:03
@vibhatha vibhatha self-assigned this Jan 19, 2024
Copy link
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

Thanks @vibhatha !
The integration failure seems related:

 Error:  Failed to execute goal org.codehaus.mojo:exec-maven-plugin:3.1.1:exec (run-java-benchmarks) on project arrow-performance: The plugin org.codehaus.mojo:exec-maven-plugin:3.1.1 requires Maven version 3.6.3 -> [Help 1]
Error:  
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:  
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginIncompatibleException
Error:  
Error:  After correcting the problems, you can resume the build with the command
Error:    mvn <goals> -rf :arrow-performance

Also, this should not be MINOR and we should create an issue for it.

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting review Awaiting review labels Jan 19, 2024
@vibhatha
Copy link
Contributor Author

@raulcd let me check this. 👍

@vibhatha vibhatha changed the title MINOR: [Java] Bump org.codehaus.mojo:exec-maven-plugin from 1.6.0 to 3.1.1 in /java V2 [Java] Bump org.codehaus.mojo:exec-maven-plugin from 1.6.0 to 3.1.1 Jan 22, 2024
@vibhatha
Copy link
Contributor Author

@raulcd is there a way to re-run the CIs.

I looked into the logs and noticed that the environment already has MAVEN 3.6.3

Error:  Failed to execute goal org.codehaus.mojo:exec-maven-plugin:3.1.1:exec (run-java-benchmarks) on project arrow-performance: The plugin org.codehaus.mojo:exec-maven-plugin:3.1.1 requires Maven version 3.6.3 -> [Help 1]
Error:  
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:  
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginIncompatibleException
Error:  
Error:  After correcting the problems, you can resume the build with the command
Error:    mvn <goals> -rf :arrow-performance
1
Error: `docker-compose --file /home/runner/work/arrow/arrow/docker-compose.yml run --rm -e ARCHERY_DEFAULT_BRANCH=main -e ARCHERY_INTEGRATION_WITH_RUST=1 conda-integration` exited with a non-zero exit code 1, see the process log above.

The docker-compose command was invoked with the following parameters:

Defaults defined in .env:
  ALMALINUX: 8
  ALPINE_LINUX: 3.16
  ARCH: amd64
  ARCH_ALIAS: x86_64
  ARCH_SHORT: amd64
  ARROW_R_DEV: TRUE
  BUILDKIT_INLINE_CACHE: 1
  CLANG_TOOLS: 14
  COMPOSE_DOCKER_CLI_BUILD: 1
  CONAN: gcc10
  CUDA: 11.2.2
  DASK: latest
  DEBIAN: 11
  DEVTOOLSET_VERSION: 
  DOCKER_BUILDKIT: 1
  DOCKER_VOLUME_PREFIX: 
  DOTNET: 7.0
  FEDORA: 38
  GCC_VERSION: 
  GO: 1.19.13
  HDFS: 3.2.1
  JDK: 8
  KARTOTHEK: latest
  LLVM: 14
  MAVEN: 3.6.3
  NODE: 18
  NUMBA: latest
  NUMPY: latest
  PANDAS: latest
  PYTHON: 3.8
  PYTHON_WHEEL_WINDOWS_IMAGE_REVISION: 2023-08-02
  R: 4.2
  REPO: apache/arrow-dev
  R_CUSTOM_CCACHE: false
  R_IMAGE: ubuntu-gcc-release
  R_ORG: rhub
  R_PRUNE_DEPS: FALSE
  R_TAG: latest
  SPARK: master
  STATICCHECK: v0.4.5
  TURBODBC: latest
  TZ: UTC
  UBUNTU: 20.04
  ULIMIT_CORE: -1

@vibhatha
Copy link
Contributor Author

@github-actions crossbow submit java

@github-actions
Copy link

Revision: e1c246622418456d6582df60db993b4a67b0166c

Submitted crossbow builds: ursacomputing/crossbow @ actions-ce9ed1dcce

Task Status
java-jars GitHub Actions
verify-rc-source-java-linux-almalinux-8-amd64 GitHub Actions
verify-rc-source-java-linux-conda-latest-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-20.04-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-22.04-amd64 GitHub Actions
verify-rc-source-java-macos-amd64 GitHub Actions

@lidavidm
Copy link
Member

I kicked the job.

@vibhatha
Copy link
Contributor Author

Thanks @lidavidm

@lidavidm
Copy link
Member

It appears it's still failing. You'll want to dig into the image, perhaps it's picking up the wrong Maven.

@vibhatha
Copy link
Contributor Author

Yeah. Will analyze this. Thanks @lidavidm

@vibhatha
Copy link
Contributor Author

@raulcd created a ticket: #39734

How can we link to this PR? 🙂

@assignUser assignUser changed the title [Java] Bump org.codehaus.mojo:exec-maven-plugin from 1.6.0 to 3.1.1 GH-39734: [Java] Bump org.codehaus.mojo:exec-maven-plugin from 1.6.0 to 3.1.1 Jan 23, 2024
@vibhatha vibhatha force-pushed the feat-bump-org.codehaus.mojo-3.1.1 branch from e1c2466 to 75448b1 Compare January 23, 2024 04:35
@github-actions github-actions bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels Jan 23, 2024
@vibhatha
Copy link
Contributor Author

Thanks @assignUser

@github-actions github-actions bot added the awaiting change review Awaiting change review label Jan 24, 2024
@github-actions
Copy link

Revision: 6ea7e1f

Submitted crossbow builds: ursacomputing/crossbow @ actions-554e5c5e99

Task Status
java-jars GitHub Actions
verify-rc-source-java-linux-almalinux-8-amd64 GitHub Actions
verify-rc-source-java-linux-conda-latest-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-20.04-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-22.04-amd64 GitHub Actions
verify-rc-source-java-macos-amd64 GitHub Actions

@vibhatha vibhatha requested a review from lidavidm January 24, 2024 03:34
@vibhatha
Copy link
Contributor Author

@github-actions crossbow submit java

@github-actions
Copy link

Revision: 4b059ac

Submitted crossbow builds: ursacomputing/crossbow @ actions-77c87708fd

Task Status
java-jars GitHub Actions
verify-rc-source-java-linux-almalinux-8-amd64 GitHub Actions
verify-rc-source-java-linux-conda-latest-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-20.04-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-22.04-amd64 GitHub Actions
verify-rc-source-java-macos-amd64 GitHub Actions

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting change review Awaiting change review labels Jan 24, 2024
@vibhatha
Copy link
Contributor Author

vibhatha commented Feb 2, 2024

@github-actions crossbow submit java

@github-actions
Copy link

github-actions bot commented Feb 2, 2024

Revision: 4b059ac

Submitted crossbow builds: ursacomputing/crossbow @ actions-000ede4aab

Task Status
java-jars GitHub Actions
verify-rc-source-java-linux-almalinux-8-amd64 GitHub Actions
verify-rc-source-java-linux-conda-latest-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-20.04-amd64 GitHub Actions
verify-rc-source-java-linux-ubuntu-22.04-amd64 GitHub Actions
verify-rc-source-java-macos-amd64 GitHub Actions

@lidavidm
Copy link
Member

lidavidm commented Feb 2, 2024

@raulcd any more comments here?

Copy link
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

Thanks!

@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 5 benchmarking runs that have been run so far on merge-commit 0fb00fd.

There was 1 benchmark result indicating a performance regression:

The full Conbench report has more details.

kou added a commit to kou/arrow that referenced this pull request Feb 8, 2024
kou added a commit that referenced this pull request Feb 8, 2024
…9993)

### Rationale for this change

GH-39696 updated Maven version but `ubuntu-docs` haven't used it yet.

### What changes are included in this PR?

Use `MAVEN` in `.env` in `ubuntu-docs`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* Closes: #39992

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
dgreiss pushed a commit to dgreiss/arrow that referenced this pull request Feb 19, 2024
…1.6.0 to 3.1.1 (apache#39696)

### Rationale for this change

This PR was created to replace apache#39374 and do the necessary changes for `org.codehaus.mojo` upgrade to take place. 

### What changes are included in this PR?

The changes to the `org.codehaus.mojo` version and an upgrade on the maven version used in the `.env`. 

### Are these changes tested?

Tested locally, but this requires a CI verification on Java.

### Are there any user-facing changes?

No

Authored-by: vibhatha <vibhatha@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
dgreiss pushed a commit to dgreiss/arrow that referenced this pull request Feb 19, 2024
…nv (apache#39993)

### Rationale for this change

apacheGH-39696 updated Maven version but `ubuntu-docs` haven't used it yet.

### What changes are included in this PR?

Use `MAVEN` in `.env` in `ubuntu-docs`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* Closes: apache#39992

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
…1.6.0 to 3.1.1 (apache#39696)

### Rationale for this change

This PR was created to replace apache#39374 and do the necessary changes for `org.codehaus.mojo` upgrade to take place. 

### What changes are included in this PR?

The changes to the `org.codehaus.mojo` version and an upgrade on the maven version used in the `.env`. 

### Are these changes tested?

Tested locally, but this requires a CI verification on Java.

### Are there any user-facing changes?

No

Authored-by: vibhatha <vibhatha@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
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.

3 participants