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