Back to all reviewers

Use descriptive names

apache/spark
Based on 3 comments
Java

Choose names that clearly convey purpose and context rather than generic or vague terms. Names should be self-documenting and provide sufficient information for developers to understand their role without additional investigation.

Naming Conventions Java

Reviewer Prompt

Choose names that clearly convey purpose and context rather than generic or vague terms. Names should be self-documenting and provide sufficient information for developers to understand their role without additional investigation.

Avoid generic prefixes like “My” or overly broad terms without context. Instead, use specific, descriptive names that indicate the entity’s purpose or functionality.

Examples of improvements:

  • MyByteArrayOutputStreamExposedBufferByteArrayOutputStream (describes the specific functionality)
  • InnerInnerJoinType (provides domain context)
  • mightContainEvenevenNumbersFoundCount (clarifies what the variable represents)

When naming decisions might seem unusual or counterintuitive, document the rationale to help future maintainers understand the choice. This is especially important when maintaining compatibility with existing APIs or external libraries.

3
Comments Analyzed
Java
Primary Language
Naming Conventions
Category

Source Discussions