Skip to content

Commit

Permalink
fix(java-client): package org.apache.pegasus.utils does not exist w…
Browse files Browse the repository at this point in the history
…hile building java client (apache#2187)

Fix apache#2188.

`idl/utils.thrift` was introduced before and has not been added into java client
to generate necessary source files which make workflow for building failed. It
should also be excluded from spotless check just like what have been done for
other thrift files, since the check would fail due to code style.
  • Loading branch information
empiredan authored Jan 22, 2025
1 parent c950ffe commit 2222230
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions java-client/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -427,6 +427,7 @@
<excludes>
<exclude>src/main/java/org/apache/pegasus/apps/*</exclude>
<exclude>src/main/java/org/apache/pegasus/replication/*</exclude>
<exclude>src/main/java/org/apache/pegasus/utils/*</exclude>
</excludes>
<googleJavaFormat>
<version>1.7</version>
Expand Down Expand Up @@ -487,6 +488,7 @@
<include>partition_split.thrift</include>
<include>rrdb.thrift</include>
<include>security.thrift</include>
<include>utils.thrift</include>
</includes>
<thriftExecutable>thrift</thriftExecutable>
<outputDirectory>${project.basedir}/src/main/java</outputDirectory>
Expand Down

0 comments on commit 2222230

Please sign in to comment.