You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2022/02/17 04:05:38 UTC

[GitHub] [kafka] C0urante opened a new pull request #11778: KAFKA-10000: Use transactional producer for config topic

C0urante opened a new pull request #11778:
URL: https://github.com/apache/kafka/pull/11778


   Implements the behavior described in [KIP-618](https://cwiki.apache.org/confluence/display/KAFKA/KIP-618%3A+Exactly-Once+Support+for+Source+Connectors#KIP618:ExactlyOnceSupportforSourceConnectors-Leaderaccesstoconfigtopic): using a transactional producer for writes to the config topic that should only be performed by the leader of the cluster.
   
   Relies on changes from:
   - https://github.com/apache/kafka/pull/11774
   - https://github.com/apache/kafka/pull/11775


-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [kafka] C0urante commented on pull request #11778: KAFKA-10000: Use transactional producer for config topic (KIP-618)

Posted by GitBox <gi...@apache.org>.
C0urante commented on pull request #11778:
URL: https://github.com/apache/kafka/pull/11778#issuecomment-1043994475


   Converting to draft until upstream PRs are reviewed.


-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [kafka] C0urante commented on a change in pull request #11778: KAFKA-10000: Use transactional producer for config topic (KIP-618)

Posted by GitBox <gi...@apache.org>.
C0urante commented on a change in pull request #11778:
URL: https://github.com/apache/kafka/pull/11778#discussion_r821155498



##########
File path: connect/runtime/src/main/java/org/apache/kafka/connect/storage/ClusterConfigState.java
##########
@@ -14,7 +14,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package org.apache.kafka.connect.runtime.distributed;
+package org.apache.kafka.connect.storage;

Review comment:
       Moved to a different package to leverage package-private fields and methods while testing other classes in this package (such as the `KafkaConfigBackingStore` class).




-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org