You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "showuon (via GitHub)" <gi...@apache.org> on 2026/06/04 12:04:46 UTC
[PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
showuon opened a new pull request, #22470:
URL: https://github.com/apache/kafka/pull/22470
Added `AGENTS.md` and a symbolic link `CLAUDE.md` file like how [Apache Airflow](https://github.com/apache/airflow) does.
--
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
Re: [PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
Posted by "showuon (via GitHub)" <gi...@apache.org>.
showuon commented on code in PR #22470:
URL: https://github.com/apache/kafka/pull/22470#discussion_r3371989107
##########
CONTRIBUTING.md:
##########
@@ -4,8 +4,27 @@
It lists steps that are required before creating a PR.
+### License
+
When you contribute code, you affirm that the contribution is your original work and that you
license the work to the project under the project's open source license. Whether or not you
state this explicitly, by submitting any copyrighted material via pull request, email, or
other means you agree to license the material under the project's open source license and
warrant that you have the legal authority to do so.
+
+### AI-Generated Contributions
+
+Based on the [ASF Generative Tooling Guidance](https://www.apache.org/legal/generative-tooling.html):
+
+- If you used AI tools in preparing your PR, please commit with the `Co-Authored-By`, `Generated-by`, or similar commit trailer.
+ ```
+ Co-Authored-By: <AI tool name and version>
+ Generated-by: <AI tool name and version>
+ ```
+ Example:
+ ```
+ KAFKA-9999: adding AGENTS.md
+
+ Co-Authored-By: Claude Sonnet 4.6 <cl...@anthropic.com>
+ ```
+- The human author remains responsible for all changes, regardless of the tools used to create them, and must certify that you wrote or otherwise have the right to submit the code you are contributing.
Review Comment:
I know what you're talking about. Let's have the discussion on the dev mailing list. At least we document something first.
--
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
Re: [PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
Posted by "showuon (via GitHub)" <gi...@apache.org>.
showuon commented on PR #22470:
URL: https://github.com/apache/kafka/pull/22470#issuecomment-4658585528
@AndrewJSchofield , thanks for the comments. PR updated in https://github.com/apache/kafka/pull/22470/commits/4e81ebeacccdb1f3a3f9d275b705bd46ab5124ca.
--
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
Re: [PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
Posted by "viktorsomogyi (via GitHub)" <gi...@apache.org>.
viktorsomogyi commented on code in PR #22470:
URL: https://github.com/apache/kafka/pull/22470#discussion_r3356554883
##########
AGENTS.md:
##########
@@ -0,0 +1,127 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients` and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|------|--------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/` | Coordinators |
+| `connect/` | Kafka Connect (api, runtime, plugins, transforms) |
+| `streams/` | Kafka Streams (+ Scala, examples, upgrade system tests) |
+| `generator/` | RPC/message code generation |
+| `tools/`, `shell/` | CLI tools and kafka metadata shell
Review Comment:
nit: Missing trailing `|`
##########
AGENTS.md:
##########
@@ -0,0 +1,127 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients` and `streams`; 17+ for other modules.
Review Comment:
Should we add generator too to this list? Although it may not be frequently contributed to, it's a top level component too.
##########
AGENTS.md:
##########
@@ -0,0 +1,127 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients` and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|------|--------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/` | Coordinators |
+| `connect/` | Kafka Connect (api, runtime, plugins, transforms) |
+| `streams/` | Kafka Streams (+ Scala, examples, upgrade system tests) |
+| `generator/` | RPC/message code generation |
+| `tools/`, `shell/` | CLI tools and kafka metadata shell
+| `examples/` | Kafka producer and consumer examples |
+| `jmh-benchmarks/` | Kafka benchmarks tests |
+| `trogdor/` | test framework |
+| `tests/` | System test harness (see `tests/README.md`) |
+| `config/` | Sample broker/controller configuration |
+| `checkstyle/` | Checkstyle rules and import-control XML |
+| `docs/` | Documentation sources |
+
+
+## Build commands
+
+```bash
+# Compile jars
+./gradlew jar
+
+# Release tarball (output under core/build/distributions/)
+./gradlew clean releaseTarGz
+
+# Publish to local Maven (skip signing for local dev)
+./gradlew -PskipSigning=true publishToMavenLocal
+
+# Regenerate RPC/message classes after branch switches
+./gradlew processMessages processTestMessages
+
+# List tasks
+./gradlew tasks
+```
+
+Module-scoped build:
+
+```bash
+./gradlew core:jar
+./gradlew clients:jar
+./gradlew :streams:testAll
+```
+
+## Test commands
+
+```bash
+# All unit + integration tests
+./gradlew test
+
+# Split suites
+./gradlew unitTest
+./gradlew integrationTest
+
+# Include flaky tests (marked @Flaky)
+./gradlew test -Pkafka.test.run.flaky=true
+
+# Re-run without considering up-to-date
+./gradlew test --rerun-tasks
+
+# Single test class
+./gradlew clients:test --tests RequestResponseTest
+
+# Single test method
+./gradlew core:test --tests kafka.api.ProducerFailureHandlingTest.testCannotSendToInternalTopic
+
+# Coverage (whole project or one module)
+./gradlew reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+./gradlew clients:reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+```
+
+**Test retries** (optional): `-PmaxTestRetries=1 -PmaxTestRetryFailures=3`
+
+**Logging**: adjust `log4j2.yaml` under `<module>/src/test/resources/` for verbose test logs.
+
+**System tests**: see [tests/README.md](tests/README.md).
+
+## Code quality
+
+Run before submitting Java changes:
+
+```bash
+./gradlew spotlessApply # fix import order / formatting
+./gradlew checkstyleMain checkstyleTest spotlessCheck
+./gradlew spotbugsMain spotbugsTest -x test # optional static analysis
+```
+
+Checkstyle reports: `<module>/build/reports/checkstyle/`.
+
+## Coding conventions
Review Comment:
We could add commit conventions that we use. The agents should start their commits with KAFKA-XXXXX or MINOR and should reference a valid Jira ticket.
We could also mention that [some packages](https://github.com/apache/kafka/blob/d559af58d8d0be113b6a96b6367a21b3e4d04197/build.gradle#L2126) contain public interfaces and modifying them or any protocol / config / metric generally require a KIP.
--
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
Re: [PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
Posted by "showuon (via GitHub)" <gi...@apache.org>.
showuon commented on PR #22470:
URL: https://github.com/apache/kafka/pull/22470#issuecomment-4646375195
@viktorsomogyi , thanks for the review. I've addressed your comments in https://github.com/apache/kafka/pull/22470/commits/5a9a36cd7cad4ec50f4f1c008c7e75f8d616cdd1 . I also added one paragraph in `CONTRIBUTING.md` to mention the rule of `AI-Generated Contributions` in https://github.com/apache/kafka/pull/22470/commits/a964135116cba2236ee4099573705e582cb21df8. Thanks.
--
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
Re: [PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
Posted by "AndrewJSchofield (via GitHub)" <gi...@apache.org>.
AndrewJSchofield commented on code in PR #22470:
URL: https://github.com/apache/kafka/pull/22470#discussion_r3379197385
##########
AGENTS.md:
##########
@@ -0,0 +1,130 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients`, `generator`, and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|------|--------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/` | Coordinators |
+| `connect/` | Kafka Connect (api, runtime, plugins, transforms) |
+| `streams/` | Kafka Streams (+ Scala, examples, upgrade system tests) |
+| `generator/` | RPC/message code generation |
+| `tools/`, `shell/` | CLI tools and kafka metadata shell |
+| `examples/` | Kafka producer and consumer examples |
+| `jmh-benchmarks/` | Kafka benchmarks tests |
+| `trogdor/` | test framework |
+| `tests/` | System test harness (see `tests/README.md`) |
+| `config/` | Sample broker/controller configuration |
+| `checkstyle/` | Checkstyle rules and import-control XML |
+| `docs/` | Documentation sources |
+
+
+## Build commands
+
+```bash
+# Compile jars
+./gradlew jar
+
+# Release tarball (output under core/build/distributions/)
+./gradlew clean releaseTarGz
+
+# Publish to local Maven (skip signing for local dev)
+./gradlew -PskipSigning=true publishToMavenLocal
+
+# Regenerate RPC/message classes after branch switches
+./gradlew processMessages processTestMessages
+
+# List tasks
+./gradlew tasks
+```
+
+Module-scoped build:
+
+```bash
+./gradlew core:jar
+./gradlew clients:jar
+./gradlew :streams:testAll
+```
+
+## Test commands
+
+```bash
+# All unit + integration tests
+./gradlew test
+
+# Split suites
+./gradlew unitTest
+./gradlew integrationTest
+
+# Include flaky tests (marked @Flaky)
+./gradlew test -Pkafka.test.run.flaky=true
+
+# Re-run without considering up-to-date
+./gradlew test --rerun-tasks
+
+# Single test class
+./gradlew clients:test --tests RequestResponseTest
+
+# Single test method
+./gradlew core:test --tests kafka.api.ProducerFailureHandlingTest.testCannotSendToInternalTopic
+
+# Coverage (whole project or one module)
+./gradlew reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+./gradlew clients:reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+```
+
+**Test retries** (optional): `-PmaxTestRetries=1 -PmaxTestRetryFailures=3`
+
+**Logging**: adjust `log4j2.yaml` under `<module>/src/test/resources/` for verbose test logs.
+
+**System tests**: see [tests/README.md](tests/README.md).
+
+## Code quality
+
+Run before submitting Java changes:
+
+```bash
+./gradlew spotlessApply # fix import order / formatting
+./gradlew checkstyleMain checkstyleTest spotlessCheck
+./gradlew spotbugsMain spotbugsTest -x test # optional static analysis
+```
+
+Checkstyle reports: `<module>/build/reports/checkstyle/`.
+
+## Coding conventions
+
+- **Commits / PR titles**: Start with `KAFKA-XXXXX`, `MINOR`, or `HOTFIX`. Use `KAFKA-XXXXX` only when there is a valid Jira ticket for the change.
+- **AI-generated contributions**: Follow the [AI-Generated Contributions](CONTRIBUTING.md#ai-generated-contributions) section in `CONTRIBUTING.md` — add a `Co-Authored-By` or `Generated-by` commit trailer for AI-assisted changes.
+- **Public API / KIP**: Changes to public client interfaces, wire protocol, broker configuration, or metrics generally require a [KIP](https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals). See the `javadoc` `include` list in [build.gradle](build.gradle) for packages treated as public API.
Review Comment:
I think all configurations generally require a KIP (broker, client, group, topic).
##########
AGENTS.md:
##########
@@ -0,0 +1,130 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients`, `generator`, and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|------|--------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/` | Coordinators |
+| `connect/` | Kafka Connect (api, runtime, plugins, transforms) |
+| `streams/` | Kafka Streams (+ Scala, examples, upgrade system tests) |
+| `generator/` | RPC/message code generation |
+| `tools/`, `shell/` | CLI tools and kafka metadata shell |
+| `examples/` | Kafka producer and consumer examples |
+| `jmh-benchmarks/` | Kafka benchmarks tests |
+| `trogdor/` | test framework |
+| `tests/` | System test harness (see `tests/README.md`) |
+| `config/` | Sample broker/controller configuration |
+| `checkstyle/` | Checkstyle rules and import-control XML |
+| `docs/` | Documentation sources |
+
+
+## Build commands
+
+```bash
+# Compile jars
+./gradlew jar
+
+# Release tarball (output under core/build/distributions/)
+./gradlew clean releaseTarGz
+
+# Publish to local Maven (skip signing for local dev)
+./gradlew -PskipSigning=true publishToMavenLocal
+
+# Regenerate RPC/message classes after branch switches
+./gradlew processMessages processTestMessages
+
+# List tasks
+./gradlew tasks
+```
+
+Module-scoped build:
+
+```bash
+./gradlew core:jar
+./gradlew clients:jar
+./gradlew :streams:testAll
+```
+
+## Test commands
+
+```bash
+# All unit + integration tests
+./gradlew test
+
+# Split suites
+./gradlew unitTest
+./gradlew integrationTest
+
+# Include flaky tests (marked @Flaky)
+./gradlew test -Pkafka.test.run.flaky=true
+
+# Re-run without considering up-to-date
+./gradlew test --rerun-tasks
+
+# Single test class
+./gradlew clients:test --tests RequestResponseTest
+
+# Single test method
+./gradlew core:test --tests kafka.api.ProducerFailureHandlingTest.testCannotSendToInternalTopic
+
+# Coverage (whole project or one module)
+./gradlew reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+./gradlew clients:reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+```
+
+**Test retries** (optional): `-PmaxTestRetries=1 -PmaxTestRetryFailures=3`
+
+**Logging**: adjust `log4j2.yaml` under `<module>/src/test/resources/` for verbose test logs.
+
+**System tests**: see [tests/README.md](tests/README.md).
+
+## Code quality
+
+Run before submitting Java changes:
+
+```bash
+./gradlew spotlessApply # fix import order / formatting
+./gradlew checkstyleMain checkstyleTest spotlessCheck
+./gradlew spotbugsMain spotbugsTest -x test # optional static analysis
+```
+
+Checkstyle reports: `<module>/build/reports/checkstyle/`.
+
+## Coding conventions
+
+- **Commits / PR titles**: Start with `KAFKA-XXXXX`, `MINOR`, or `HOTFIX`. Use `KAFKA-XXXXX` only when there is a valid Jira ticket for the change.
+- **AI-generated contributions**: Follow the [AI-Generated Contributions](CONTRIBUTING.md#ai-generated-contributions) section in `CONTRIBUTING.md` — add a `Co-Authored-By` or `Generated-by` commit trailer for AI-assisted changes.
+- **Public API / KIP**: Changes to public client interfaces, wire protocol, broker configuration, or metrics generally require a [KIP](https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals). See the `javadoc` `include` list in [build.gradle](build.gradle) for packages treated as public API.
+- **License header**: New source files need the standard ASF license header (see existing files in the same module).
+- **Checkstyle**: Enforced on main and test sources; rules in `checkstyle/`. Import order is checked via Spotless.
+- **Generated code**: Do not hand-edit generated RPC/message sources. Regenerate with `processMessages` / `processTestMessages`. See [clients/src/main/resources/common/message/README.md](clients/src/main/resources/common/message/README.md).
+- **Scope**: Prefer targeted unit tests in the module you change; integration/system tests for cross-component behavior.
+- **Compatibility**: Client and Streams modules target older Java bytecode; avoid APIs that break those release levels.
Review Comment:
I think it is appropriate to state the Java versions which are supported here.
##########
AGENTS.md:
##########
@@ -0,0 +1,130 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients`, `generator`, and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|------|--------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/` | Coordinators |
Review Comment:
coordinator-common too.
##########
AGENTS.md:
##########
@@ -0,0 +1,130 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients`, `generator`, and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|------|--------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/` | Coordinators |
+| `connect/` | Kafka Connect (api, runtime, plugins, transforms) |
+| `streams/` | Kafka Streams (+ Scala, examples, upgrade system tests) |
+| `generator/` | RPC/message code generation |
+| `tools/`, `shell/` | CLI tools and kafka metadata shell |
+| `examples/` | Kafka producer and consumer examples |
+| `jmh-benchmarks/` | Kafka benchmarks tests |
+| `trogdor/` | test framework |
+| `tests/` | System test harness (see `tests/README.md`) |
+| `config/` | Sample broker/controller configuration |
+| `checkstyle/` | Checkstyle rules and import-control XML |
+| `docs/` | Documentation sources |
+
+
+## Build commands
+
+```bash
+# Compile jars
+./gradlew jar
+
+# Release tarball (output under core/build/distributions/)
+./gradlew clean releaseTarGz
+
+# Publish to local Maven (skip signing for local dev)
+./gradlew -PskipSigning=true publishToMavenLocal
+
+# Regenerate RPC/message classes after branch switches
+./gradlew processMessages processTestMessages
+
+# List tasks
+./gradlew tasks
+```
+
+Module-scoped build:
+
+```bash
+./gradlew core:jar
+./gradlew clients:jar
+./gradlew :streams:testAll
+```
+
+## Test commands
+
+```bash
+# All unit + integration tests
+./gradlew test
+
+# Split suites
+./gradlew unitTest
+./gradlew integrationTest
+
+# Include flaky tests (marked @Flaky)
+./gradlew test -Pkafka.test.run.flaky=true
+
+# Re-run without considering up-to-date
+./gradlew test --rerun-tasks
+
+# Single test class
+./gradlew clients:test --tests RequestResponseTest
+
+# Single test method
+./gradlew core:test --tests kafka.api.ProducerFailureHandlingTest.testCannotSendToInternalTopic
+
+# Coverage (whole project or one module)
+./gradlew reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+./gradlew clients:reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+```
+
+**Test retries** (optional): `-PmaxTestRetries=1 -PmaxTestRetryFailures=3`
+
+**Logging**: adjust `log4j2.yaml` under `<module>/src/test/resources/` for verbose test logs.
+
+**System tests**: see [tests/README.md](tests/README.md).
+
+## Code quality
+
+Run before submitting Java changes:
+
+```bash
+./gradlew spotlessApply # fix import order / formatting
+./gradlew checkstyleMain checkstyleTest spotlessCheck
+./gradlew spotbugsMain spotbugsTest -x test # optional static analysis
+```
+
+Checkstyle reports: `<module>/build/reports/checkstyle/`.
+
+## Coding conventions
+
+- **Commits / PR titles**: Start with `KAFKA-XXXXX`, `MINOR`, or `HOTFIX`. Use `KAFKA-XXXXX` only when there is a valid Jira ticket for the change.
+- **AI-generated contributions**: Follow the [AI-Generated Contributions](CONTRIBUTING.md#ai-generated-contributions) section in `CONTRIBUTING.md` — add a `Co-Authored-By` or `Generated-by` commit trailer for AI-assisted changes.
+- **Public API / KIP**: Changes to public client interfaces, wire protocol, broker configuration, or metrics generally require a [KIP](https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals). See the `javadoc` `include` list in [build.gradle](build.gradle) for packages treated as public API.
+- **License header**: New source files need the standard ASF license header (see existing files in the same module).
+- **Checkstyle**: Enforced on main and test sources; rules in `checkstyle/`. Import order is checked via Spotless.
+- **Generated code**: Do not hand-edit generated RPC/message sources. Regenerate with `processMessages` / `processTestMessages`. See [clients/src/main/resources/common/message/README.md](clients/src/main/resources/common/message/README.md).
+- **Scope**: Prefer targeted unit tests in the module you change; integration/system tests for cross-component behavior.
+- **Compatibility**: Client and Streams modules target older Java bytecode; avoid APIs that break those release levels.
Review Comment:
I read this file backwards. I see that you've already included this information. :)
##########
AGENTS.md:
##########
@@ -0,0 +1,130 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients`, `generator`, and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|------|--------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling |
Review Comment:
I would say that this module is gradually being removed, along with Scala code.
##########
AGENTS.md:
##########
@@ -0,0 +1,130 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients`, `generator`, and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|------|--------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/` | Coordinators |
+| `connect/` | Kafka Connect (api, runtime, plugins, transforms) |
+| `streams/` | Kafka Streams (+ Scala, examples, upgrade system tests) |
+| `generator/` | RPC/message code generation |
+| `tools/`, `shell/` | CLI tools and kafka metadata shell |
+| `examples/` | Kafka producer and consumer examples |
+| `jmh-benchmarks/` | Kafka benchmarks tests |
+| `trogdor/` | test framework |
+| `tests/` | System test harness (see `tests/README.md`) |
+| `config/` | Sample broker/controller configuration |
+| `checkstyle/` | Checkstyle rules and import-control XML |
+| `docs/` | Documentation sources |
+
Review Comment:
And `docker/`
--
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
Re: [PR] MINOR: adding AGENTS.md [kafka]
Posted by "showuon (via GitHub)" <gi...@apache.org>.
showuon commented on PR #22470:
URL: https://github.com/apache/kafka/pull/22470#issuecomment-4737736572
Rebased with the latest trunk with the [new added AGENTS.md in trunk](https://github.com/apache/kafka/blob/trunk/AGENTS.md).
@AndrewJSchofield , do you want to have any other comments? Thank you.
--
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
Re: [PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
Posted by "showuon (via GitHub)" <gi...@apache.org>.
showuon commented on code in PR #22470:
URL: https://github.com/apache/kafka/pull/22470#discussion_r3379487881
##########
AGENTS.md:
##########
@@ -0,0 +1,130 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients`, `generator`, and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|------|--------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/` | Coordinators |
+| `connect/` | Kafka Connect (api, runtime, plugins, transforms) |
+| `streams/` | Kafka Streams (+ Scala, examples, upgrade system tests) |
+| `generator/` | RPC/message code generation |
+| `tools/`, `shell/` | CLI tools and kafka metadata shell |
+| `examples/` | Kafka producer and consumer examples |
+| `jmh-benchmarks/` | Kafka benchmarks tests |
+| `trogdor/` | test framework |
+| `tests/` | System test harness (see `tests/README.md`) |
+| `config/` | Sample broker/controller configuration |
+| `checkstyle/` | Checkstyle rules and import-control XML |
+| `docs/` | Documentation sources |
+
+
+## Build commands
+
+```bash
+# Compile jars
+./gradlew jar
+
+# Release tarball (output under core/build/distributions/)
+./gradlew clean releaseTarGz
+
+# Publish to local Maven (skip signing for local dev)
+./gradlew -PskipSigning=true publishToMavenLocal
+
+# Regenerate RPC/message classes after branch switches
+./gradlew processMessages processTestMessages
+
+# List tasks
+./gradlew tasks
+```
+
+Module-scoped build:
+
+```bash
+./gradlew core:jar
+./gradlew clients:jar
+./gradlew :streams:testAll
+```
+
+## Test commands
+
+```bash
+# All unit + integration tests
+./gradlew test
+
+# Split suites
+./gradlew unitTest
+./gradlew integrationTest
+
+# Include flaky tests (marked @Flaky)
+./gradlew test -Pkafka.test.run.flaky=true
+
+# Re-run without considering up-to-date
+./gradlew test --rerun-tasks
+
+# Single test class
+./gradlew clients:test --tests RequestResponseTest
+
+# Single test method
+./gradlew core:test --tests kafka.api.ProducerFailureHandlingTest.testCannotSendToInternalTopic
+
+# Coverage (whole project or one module)
+./gradlew reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+./gradlew clients:reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+```
+
+**Test retries** (optional): `-PmaxTestRetries=1 -PmaxTestRetryFailures=3`
+
+**Logging**: adjust `log4j2.yaml` under `<module>/src/test/resources/` for verbose test logs.
+
+**System tests**: see [tests/README.md](tests/README.md).
+
+## Code quality
+
+Run before submitting Java changes:
+
+```bash
+./gradlew spotlessApply # fix import order / formatting
+./gradlew checkstyleMain checkstyleTest spotlessCheck
+./gradlew spotbugsMain spotbugsTest -x test # optional static analysis
+```
+
+Checkstyle reports: `<module>/build/reports/checkstyle/`.
+
+## Coding conventions
+
+- **Commits / PR titles**: Start with `KAFKA-XXXXX`, `MINOR`, or `HOTFIX`. Use `KAFKA-XXXXX` only when there is a valid Jira ticket for the change.
+- **AI-generated contributions**: Follow the [AI-Generated Contributions](CONTRIBUTING.md#ai-generated-contributions) section in `CONTRIBUTING.md` — add a `Co-Authored-By` or `Generated-by` commit trailer for AI-assisted changes.
+- **Public API / KIP**: Changes to public client interfaces, wire protocol, broker configuration, or metrics generally require a [KIP](https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals). See the `javadoc` `include` list in [build.gradle](build.gradle) for packages treated as public API.
+- **License header**: New source files need the standard ASF license header (see existing files in the same module).
+- **Checkstyle**: Enforced on main and test sources; rules in `checkstyle/`. Import order is checked via Spotless.
+- **Generated code**: Do not hand-edit generated RPC/message sources. Regenerate with `processMessages` / `processTestMessages`. See [clients/src/main/resources/common/message/README.md](clients/src/main/resources/common/message/README.md).
+- **Scope**: Prefer targeted unit tests in the module you change; integration/system tests for cross-component behavior.
+- **Compatibility**: Client and Streams modules target older Java bytecode; avoid APIs that break those release levels.
Review Comment:
Thanks. :)
--
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
Re: [PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
Posted by "viktorsomogyi (via GitHub)" <gi...@apache.org>.
viktorsomogyi commented on code in PR #22470:
URL: https://github.com/apache/kafka/pull/22470#discussion_r3371899382
##########
CONTRIBUTING.md:
##########
@@ -4,8 +4,27 @@
It lists steps that are required before creating a PR.
+### License
+
When you contribute code, you affirm that the contribution is your original work and that you
license the work to the project under the project's open source license. Whether or not you
state this explicitly, by submitting any copyrighted material via pull request, email, or
other means you agree to license the material under the project's open source license and
warrant that you have the legal authority to do so.
+
+### AI-Generated Contributions
+
+Based on the [ASF Generative Tooling Guidance](https://www.apache.org/legal/generative-tooling.html):
+
+- If you used AI tools in preparing your PR, please commit with the `Co-Authored-By`, `Generated-by`, or similar commit trailer.
+ ```
+ Co-Authored-By: <AI tool name and version>
+ Generated-by: <AI tool name and version>
+ ```
+ Example:
+ ```
+ KAFKA-9999: adding AGENTS.md
+
+ Co-Authored-By: Claude Sonnet 4.6 <cl...@anthropic.com>
+ ```
+- The human author remains responsible for all changes, regardless of the tools used to create them, and must certify that you wrote or otherwise have the right to submit the code you are contributing.
Review Comment:
A more formal version of this is the DCO: https://github.com/apps/dco. Lightweight, but it makes the contributor very aware of that they need to attest that they're the author (and there will be no room for "didn't read it").
I don't think we should expand the scope of this PR necessarily, but I think it's better, more formal than the documentation only as it requires an explicit agreement from the contributor on the PR.
--
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
Re: [PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
Posted by "AndrewJSchofield (via GitHub)" <gi...@apache.org>.
AndrewJSchofield commented on code in PR #22470:
URL: https://github.com/apache/kafka/pull/22470#discussion_r3387366485
##########
AGENTS.md:
##########
@@ -0,0 +1,131 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients`, `generator`, and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|--------------------------------------------------------------------------|----------------------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling (gradually being removed along with Scala code) |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/`, `coordinator-common` | Coordinators |
Review Comment:
nit: `coordinator-common/` with a trailing '/' please.
--
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
Re: [PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
Posted by "showuon (via GitHub)" <gi...@apache.org>.
showuon commented on code in PR #22470:
URL: https://github.com/apache/kafka/pull/22470#discussion_r3379663830
##########
AGENTS.md:
##########
@@ -0,0 +1,130 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients`, `generator`, and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|------|--------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/` | Coordinators |
+| `connect/` | Kafka Connect (api, runtime, plugins, transforms) |
+| `streams/` | Kafka Streams (+ Scala, examples, upgrade system tests) |
+| `generator/` | RPC/message code generation |
+| `tools/`, `shell/` | CLI tools and kafka metadata shell |
+| `examples/` | Kafka producer and consumer examples |
+| `jmh-benchmarks/` | Kafka benchmarks tests |
+| `trogdor/` | test framework |
+| `tests/` | System test harness (see `tests/README.md`) |
+| `config/` | Sample broker/controller configuration |
+| `checkstyle/` | Checkstyle rules and import-control XML |
+| `docs/` | Documentation sources |
+
+
+## Build commands
+
+```bash
+# Compile jars
+./gradlew jar
+
+# Release tarball (output under core/build/distributions/)
+./gradlew clean releaseTarGz
+
+# Publish to local Maven (skip signing for local dev)
+./gradlew -PskipSigning=true publishToMavenLocal
+
+# Regenerate RPC/message classes after branch switches
+./gradlew processMessages processTestMessages
+
+# List tasks
+./gradlew tasks
+```
+
+Module-scoped build:
+
+```bash
+./gradlew core:jar
+./gradlew clients:jar
+./gradlew :streams:testAll
+```
+
+## Test commands
+
+```bash
+# All unit + integration tests
+./gradlew test
+
+# Split suites
+./gradlew unitTest
+./gradlew integrationTest
+
+# Include flaky tests (marked @Flaky)
+./gradlew test -Pkafka.test.run.flaky=true
+
+# Re-run without considering up-to-date
+./gradlew test --rerun-tasks
+
+# Single test class
+./gradlew clients:test --tests RequestResponseTest
+
+# Single test method
+./gradlew core:test --tests kafka.api.ProducerFailureHandlingTest.testCannotSendToInternalTopic
+
+# Coverage (whole project or one module)
+./gradlew reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+./gradlew clients:reportCoverage -PenableTestCoverage=true -Dorg.gradle.parallel=false
+```
+
+**Test retries** (optional): `-PmaxTestRetries=1 -PmaxTestRetryFailures=3`
+
+**Logging**: adjust `log4j2.yaml` under `<module>/src/test/resources/` for verbose test logs.
+
+**System tests**: see [tests/README.md](tests/README.md).
+
+## Code quality
+
+Run before submitting Java changes:
+
+```bash
+./gradlew spotlessApply # fix import order / formatting
+./gradlew checkstyleMain checkstyleTest spotlessCheck
+./gradlew spotbugsMain spotbugsTest -x test # optional static analysis
+```
+
+Checkstyle reports: `<module>/build/reports/checkstyle/`.
+
+## Coding conventions
+
+- **Commits / PR titles**: Start with `KAFKA-XXXXX`, `MINOR`, or `HOTFIX`. Use `KAFKA-XXXXX` only when there is a valid Jira ticket for the change.
+- **AI-generated contributions**: Follow the [AI-Generated Contributions](CONTRIBUTING.md#ai-generated-contributions) section in `CONTRIBUTING.md` — add a `Co-Authored-By` or `Generated-by` commit trailer for AI-assisted changes.
+- **Public API / KIP**: Changes to public client interfaces, wire protocol, broker configuration, or metrics generally require a [KIP](https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals). See the `javadoc` `include` list in [build.gradle](build.gradle) for packages treated as public API.
+- **License header**: New source files need the standard ASF license header (see existing files in the same module).
+- **Checkstyle**: Enforced on main and test sources; rules in `checkstyle/`. Import order is checked via Spotless.
+- **Generated code**: Do not hand-edit generated RPC/message sources. Regenerate with `processMessages` / `processTestMessages`. See [clients/src/main/resources/common/message/README.md](clients/src/main/resources/common/message/README.md).
+- **Scope**: Prefer targeted unit tests in the module you change; integration/system tests for cross-component behavior.
+- **Compatibility**: Client and Streams modules target older Java bytecode; avoid APIs that break those release levels.
Review Comment:
I still added it anyway.
--
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
Re: [PR] MINOR: adding AGENTS.md and CLAUDE.md [kafka]
Posted by "showuon (via GitHub)" <gi...@apache.org>.
showuon commented on code in PR #22470:
URL: https://github.com/apache/kafka/pull/22470#discussion_r3387709888
##########
AGENTS.md:
##########
@@ -0,0 +1,131 @@
+# Apache Kafka — Agent Guide
+
+This file provides guidance for AI coding agents working with the Apache Kafka codebase.
+For human contributors, see [README.md](README.md) and [CONTRIBUTING.md](CONTRIBUTING.md).
+
+## Project overview
+
+Apache Kafka is a distributed event streaming platform. This repo is a **Gradle**
+build in **Java** and **Scala 2.13**.
+
+- **Supported JDK**: 11+ for `clients`, `generator`, and `streams`; 17+ for other modules.
+- **Build/test JDK**: Java 17 and 25.
+- **Build tool**: Gradle, wrapper `./gradlew` is recommended.
+
+## Repository layout (high level)
+
+| Path | Purpose |
+|--------------------------------------------------------------------------|----------------------------------------------------------------------------------------------------|
+| `clients/` | Producer, consumer, admin client APIs and protocol messages |
+| `core/` | Broker runtime, log, replication, request handling (gradually being removed along with Scala code) |
+| `server/`, `server-common/` | Broker/server components and shared server code |
+| `metadata/`, `raft/` | KRaft metadata layer and Raft implementation |
+| `storage/`, `storage/api/` | Log segments, checkpoints, tiered storage APIs |
+| `group-coordinator/`, `share-coordinator/`, `transaction-coordinator/`, `coordinator-common` | Coordinators |
Review Comment:
Ah, nice catch! Updated in https://github.com/apache/kafka/pull/22470/commits/e4ae8bc6ad74e9863afe358f96993a0a02787826. Thanks.
--
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
Re: [PR] MINOR: adding AGENTS.md [kafka]
Posted by "showuon (via GitHub)" <gi...@apache.org>.
showuon merged PR #22470:
URL: https://github.com/apache/kafka/pull/22470
--
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