Skip to content

Commit

Permalink
apacheGH-43432: [Java][Packaging] Clean up java-jars job (apache#43431)
Browse files Browse the repository at this point in the history
### Rationale for this change
1) Remove maven module references
2) Fix warning in java-jars job:

```
Warning:  Some problems were encountered while building the effective model for org.apache.arrow:arrow-bom:pom:18.0.0-SNAPSHOT
Warning:  'parent.relativePath' of POM org.apache.arrow:arrow-bom:18.0.0-SNAPSHOT (/Users/runner/work/crossbow/crossbow/arrow/java/bom/pom.xml) points at org.apache.arrow:arrow-java-root instead of org.apache:apache, please verify your project structure @ line 23, column 11
Warning:  
Warning:  It is highly recommended to fix these problems because they threaten the stability of your build.
Warning:  
Warning:  For this reason, future Maven versions might no longer support building such malformed projects.
```

### What changes are included in this PR?

* Delete `mvn versions:set` for removed maven module
* Add empty relativePath to Arrow BOM, so it doesn't use arrow-java-root

### Are these changes tested?

java-jars CI job

### Are there any user-facing changes?

No
* GitHub Issue: apache#43432

Authored-by: Dane Pitkin <[email protected]>
Signed-off-by: David Li <[email protected]>
  • Loading branch information
danepitkin authored Jul 29, 2024
1 parent 07a8d7d commit c980ebf
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 0 additions & 1 deletion dev/tasks/java-jars/github.yml
Original file line number Diff line number Diff line change
Expand Up @@ -255,7 +255,6 @@ jobs:
pushd arrow/java
mvn versions:set -DnewVersion={{ arrow.no_rc_snapshot_version }}
mvn versions:set -DnewVersion={{ arrow.no_rc_snapshot_version }} -f bom
mvn versions:set -DnewVersion={{ arrow.no_rc_snapshot_version }} -f maven
popd
arrow/ci/scripts/java_full_build.sh \
$GITHUB_WORKSPACE/arrow \
Expand Down
1 change: 1 addition & 0 deletions java/bom/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ under the License.
<groupId>org.apache</groupId>
<artifactId>apache</artifactId>
<version>33</version>
<relativePath></relativePath>
</parent>

<groupId>org.apache.arrow</groupId>
Expand Down

0 comments on commit c980ebf

Please sign in to comment.