chia7712 merged PR #16610:
URL: https://github.com/apache/kafka/pull/16610
--
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: jira-unsubscr...@kafka.apache
mimaison commented on PR #16610:
URL: https://github.com/apache/kafka/pull/16610#issuecomment-2239252863
My bad. Fixed
--
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 un
chia7712 commented on PR #16610:
URL: https://github.com/apache/kafka/pull/16610#issuecomment-2238844809
@mimaison please fix the build error on scala 2.12
--
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
mimaison commented on code in PR #16610:
URL: https://github.com/apache/kafka/pull/16610#discussion_r1682908931
##
metadata/src/main/java/org/apache/kafka/metadata/properties/MetaPropertiesEnsemble.java:
##
@@ -101,9 +101,7 @@ public static class Loader {
private Option
chia7712 commented on code in PR #16610:
URL: https://github.com/apache/kafka/pull/16610#discussion_r1682639234
##
metadata/src/main/java/org/apache/kafka/metadata/properties/MetaPropertiesEnsemble.java:
##
@@ -101,9 +101,7 @@ public static class Loader {
private Option
mimaison opened a new pull request, #16610:
URL: https://github.com/apache/kafka/pull/16610
### Committer Checklist (excluded from commit message)
- [ ] Verify design and implementation
- [ ] Verify test coverage and CI build status
- [ ] Verify documentation (including upgr
chia7712 merged PR #15806:
URL: https://github.com/apache/kafka/pull/15806
--
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: jira-unsubscr...@kafka.apache
chia7712 commented on PR #15806:
URL: https://github.com/apache/kafka/pull/15806#issuecomment-2078233197
```
./gradlew cleanTest :streams:test --tests
StreamsAssignmentScaleTest.testHighAvailabilityTaskAssignorManyStandbys
:tools:test --tests
MetadataQuorumCommandTest.testDescribeQuorum
mimaison opened a new pull request, #15806:
URL: https://github.com/apache/kafka/pull/15806
### Committer Checklist (excluded from commit message)
- [ ] Verify design and implementation
- [ ] Verify test coverage and CI build status
- [ ] Verify documentation (including upgr
mimaison merged PR #14734:
URL: https://github.com/apache/kafka/pull/14734
--
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: jira-unsubscr...@kafka.apache
mimaison commented on PR #14734:
URL: https://github.com/apache/kafka/pull/14734#issuecomment-1809735473
The rebuild has a different set of failures and
`:streams:upgrade-system-tests-23:test` was fine across all pipelines. I'll
merge this in trunk. Thanks for the review!
--
This is an a
jlprat commented on PR #14734:
URL: https://github.com/apache/kafka/pull/14734#issuecomment-1808282654
@mimaison I also highly doubt this PR is the cause, but it would be good to
know if the test was just flaky or something went wrong in previous PRs
--
This is an automated message from t
mimaison commented on PR #14734:
URL: https://github.com/apache/kafka/pull/14734#issuecomment-1808279317
The PR does not touch streams and it builds fine locally with Java 21 and
Scala 2.13. To be extra safe I kicked a rebuild:
https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-1473
jlprat commented on PR #14734:
URL: https://github.com/apache/kafka/pull/14734#issuecomment-1808233060
Navigating to
https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14734/1/pipeline/
I see an error:
> FAILURE: Build failed with an exception.
>
mimaison commented on PR #14734:
URL: https://github.com/apache/kafka/pull/14734#issuecomment-1808226806
> LGTM then, as long as the system tests for 2.3 turns green somehow (or
it's accepted that is flaky)
What do you mean?
--
This is an automated message from the Apache Git Servi
mimaison commented on code in PR #14734:
URL: https://github.com/apache/kafka/pull/14734#discussion_r1391134864
##
metadata/src/main/java/org/apache/kafka/metadata/placement/StripedReplicaPlacer.java:
##
@@ -33,89 +33,86 @@
/**
* The striped replica placer.
- *
- *
- * GOAL
mimaison commented on code in PR #14734:
URL: https://github.com/apache/kafka/pull/14734#discussion_r1391134127
##
metadata/src/main/java/org/apache/kafka/metadata/placement/StripedReplicaPlacer.java:
##
@@ -33,89 +33,86 @@
/**
* The striped replica placer.
- *
- *
- * GOAL
mimaison commented on PR #14734:
URL: https://github.com/apache/kafka/pull/14734#issuecomment-1808184173
Thanks @jlprat for taking a look. In most javadoc blocks we don't close
`` tags, so I did that to be consistent.
--
This is an automated message from the Apache Git Service.
To respon
jlprat commented on code in PR #14734:
URL: https://github.com/apache/kafka/pull/14734#discussion_r1390959569
##
metadata/src/main/java/org/apache/kafka/metadata/placement/StripedReplicaPlacer.java:
##
@@ -33,89 +33,86 @@
/**
* The striped replica placer.
- *
- *
- * GOALS
mimaison opened a new pull request, #14734:
URL: https://github.com/apache/kafka/pull/14734
- Remove unused code, suppression
- Simplify/fix test assertions
- Javadoc cleanups
### Committer Checklist (excluded from commit message)
- [ ] Verify design and implementation
- [
20 matches
Mail list logo