dlmarion commented on code in PR #3203:
URL: https://github.com/apache/accumulo/pull/3203#discussion_r1117066650
##########
.github/workflows/maven.yaml:
##########
@@ -110,6 +109,12 @@ jobs:
run: mvn -B -V -e -ntp "-Dstyle.color=always" -DskipFormat ${{
matrix.profile.args }}
env:
MAVEN_OPTS: -Djansi.force=true
+ - name: Build with Maven (ZooKeeper compatibility build)
+ if: ${{ github.base_ref != 'main' }}
Review Comment:
The reason for this change is that #3200 uses newer ZooKeeper API objects,
so this check fails. #3200 isn't merged, but if it is, then this is an issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
