ijuma commented on PR #13059:
URL: https://github.com/apache/kafka/pull/13059#issuecomment-1383078896
@showuon Are you going to merge this one?
--
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
ijuma commented on PR #13078:
URL: https://github.com/apache/kafka/pull/13078#issuecomment-1383076727
> The current approach guarantees that size is strongly consistent with the
data stored in the map.
That is not required for a metric though.
That said, I didn't look at the
ijuma commented on code in PR #13099:
URL: https://github.com/apache/kafka/pull/13099#discussion_r1070520573
##
clients/src/main/java/org/apache/kafka/common/utils/Utils.java:
##
@@ -1496,4 +1496,32 @@ public static String replaceSuffix(String str, String
oldSuffix, String
ijuma commented on PR #13102:
URL: https://github.com/apache/kafka/pull/13102#issuecomment-1383075297
@jsancio @hachikuji Was there a reason to have this field or was it added
accidentally?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
ijuma commented on code in PR #13105:
URL: https://github.com/apache/kafka/pull/13105#discussion_r1070519853
##
clients/src/main/java/org/apache/kafka/clients/producer/KafkaProducer.java:
##
@@ -1472,7 +1472,7 @@ public boolean isDone() {
/**
* Callbacks that are
ijuma merged PR #13109:
URL: https://github.com/apache/kafka/pull/13109
--
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:
ijuma opened a new pull request, #13121:
URL: https://github.com/apache/kafka/pull/13121
Most were converted not to use PowerMock, but some no longer exist.
### Committer Checklist (excluded from commit message)
- [ ] Verify design and implementation
- [ ] Verify test coverage
yashmayya opened a new pull request, #13120:
URL: https://github.com/apache/kafka/pull/13120
Some common themes:
- Replace blank lines (which are ignored) with `` to separate paragraphs.
For those wondering about the omitted closing `` tags, this is actually the
recommended style
vamossagar12 commented on PR #13095:
URL: https://github.com/apache/kafka/pull/13095#issuecomment-1382883307
https://github.com/apache/kafka/pull/13095#discussion_r1069689088
I looked at other classes in tool and saw arg4jparse being used in them and
assumed that this is the
vamossagar12 commented on code in PR #13095:
URL: https://github.com/apache/kafka/pull/13095#discussion_r1070379239
##
tools/src/main/java/org/apache/kafka/tools/EndToEndLatency.java:
##
@@ -0,0 +1,309 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or
[
https://issues.apache.org/jira/browse/KAFKA-14624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17676938#comment-17676938
]
Balaji Rao commented on KAFKA-14624:
{quote}
Thanks for reporting this issue – will need to think
[
https://issues.apache.org/jira/browse/KAFKA-14624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17676934#comment-17676934
]
Matthias J. Sax commented on KAFKA-14624:
-
Thanks for reporting this issue – will need to think
Balaji Rao created KAFKA-14624:
--
Summary: State restoration is broken with standby tasks and
cache-enabled stores in processor API
Key: KAFKA-14624
URL: https://issues.apache.org/jira/browse/KAFKA-14624
13 matches
Mail list logo