You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by lh...@apache.org on 2026/06/05 08:51:44 UTC
(pulsar) branch master updated: [improve][doc] Add coding guideline for fully qualified class names (#25928)
This is an automated email from the ASF dual-hosted git repository.
lhotari pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new cec32df5778 [improve][doc] Add coding guideline for fully qualified class names (#25928)
cec32df5778 is described below
commit cec32df5778cc94147e69bd22a3023be9612fb0f
Author: Ruimin MA <ma...@gmail.com>
AuthorDate: Fri Jun 5 16:51:37 2026 +0800
[improve][doc] Add coding guideline for fully qualified class names (#25928)
---
AGENTS.md | 2 ++
CODING.md | 2 ++
2 files changed, 4 insertions(+)
diff --git a/AGENTS.md b/AGENTS.md
index e6479cf7229..40aa15f98ae 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -56,6 +56,8 @@ A few rules matter specifically when an AI tool makes the change, on top of the
these outward-facing actions — see *Licensing and provenance* above.
- **A clean local run is weak evidence for concurrency/data race fixes** (timing- and
platform-dependent). See [`CODING.md`](CODING.md#reproducing-concurrency--memory-visibility-bugs).
+- **Follow Java style guidance.** For Java conventions, including imports over fully qualified class
+ names, follow [`CODING.md`](CODING.md#style).
## Critical rules
diff --git a/CODING.md b/CODING.md
index 6623e342e75..af05622f592 100644
--- a/CODING.md
+++ b/CODING.md
@@ -13,6 +13,8 @@ for the agent-specific guardrails on top of it.
- No `@author` tags in Javadoc.
- Every `TODO` must reference a GitHub issue, e.g. `// TODO: https://github.com/apache/pulsar/issues/XXXX`.
- Checkstyle config: `buildtools/src/main/resources/pulsar/checkstyle.xml`. Lombok is enabled.
+- Prefer imports over fully qualified class names in code. Use a fully qualified class name only when
+ needed to disambiguate a name collision that imports cannot resolve.
## Logging