Skip to content

[8.2] Resolve javac warnings in extensions-sfp module #2035

Closed
Task
#2345
#2109
Closed
[8.2] Resolve javac warnings in extensions-sfp module#2035
Task
#2345
#2109
Labels
8.2This issue found in 8.2 releasep8priority 8tech-debttech debt
Milestone
8.2

Description

@vijaya-boddipudi
vijaya-boddipudi
opened on Aug 5, 2026

Description
The extensions-sfp module currently completes its build successfully, but still reports Java compiler (javac) and/or Javadoc source warnings that should be cleaned up to improve code quality and maintain a warning-free build.

Cleaning these issues will improve API documentation consistency and help maintain overall code quality.

Module

extensions-sfp

Build Summary

  • Module: extensions-sfp
  • Build Status: SUCCESS
  • Reactor Index: 35
  • Total Diagnostics: 100
  • Javac Warnings: 100 (unique: 95)
  • Javac Errors: 0
  • Javadoc Warnings: 0
  • Javadoc Errors: 0
  • Analysis Warnings: 0
  • Analysis Errors: 0

Expected Result

  • The module should build without any javac or javadoc warnings or errors.
  • All source files should compile cleanly under JDK 21 with the project's -Xlint:all settings.
  • Public classes, interfaces, methods, constructors, fields, and parameters should contain complete and valid Javadoc.

Javadoc generation should complete cleanly without warnings.

Actual Result

The module builds with status SUCCESS, but 100 diagnostic message(s) are reported during compilation.

Acceptance Criteria

  • Resolve all javac and javadoc warnings in the extensions-sfp module.
  • Ensure all public APIs have complete and valid Javadoc.
  • Verify the module builds successfully with zero javac errors, zero javadoc errors, and zero javadoc warnings.
  • Confirm no regressions are introduced during cleanup.

Documents to refer and follow:
java-api-writing-specs.md
javadoc-checklist.md
javadoc-spec-summary.md
javadoc-tool-architecture.md
javadoc-tool-comments.md

Residual after batch 1 (2026-08-07)

Agent progress (night-issue-prs)

Slice Issue Status PR Notes Updated
batch 1 calendar + content list helpers #2035 pr_opened #2322 real generics; project Xlint zero 2026-08-07T19:24:48Z
residual site-folder / remaining suppressions #2323 open ~150 diags under full strip 2026-08-07T19:24:48Z

Activity

  1. vijaya-boddipudi
    added this to the 8.2 milestone on Aug 5, 2026
  2. vijaya-boddipudi
    self-assigned this
    on Aug 5, 2026
  3. vijaya-boddipudi
    added
    tech-debttech debt
    8.2This issue found in 8.2 release
    on Aug 5, 2026
  4. vijaya-boddipudi
    added the
    Task
    issue type on Aug 5, 2026
  5. natechadwick
    added a commit that references this issue on Aug 6, 2026

    fix(extensions-sfp): suppress javac warnings in fastforward SFP module (

    c65a18e
  6. natechadwick
    closed this as completedin #2109on Aug 6, 2026

  7. vijaya-boddipudi commented on Aug 6, 2026

    @vijaya-boddipudi
    vijaya-boddipudi
    on Aug 6, 2026
    CollaboratorAuthor

    Status update (2026-08-06): The original PR for this issue (#2109) used @SuppressWarnings annotations to silence the warnings. Per the team's current strategy, those suppressions are being replaced with real fixes — parameterising raw collections (List → List<T>), qualifying static method calls by class name, and removing redundant casts. The original PR has already been merged, so this issue is being reopened and a follow-up fix/rawtypes PR will be opened shortly to complete the actual remediation.

  8. vijaya-boddipudi
    reopened this on Aug 6, 2026
  9. vijaya-boddipudi
    added a parent issue on Aug 6, 2026
  10. natechadwick
    added a commit that references this issue on Aug 7, 2026

    fix(extensions-sfp): real fixes for static-method and raw type warnin…

    7730e76
  11. vijaya-boddipudi
    removed their assignment
    on Aug 7, 2026
  12. natechadwick-intsof
    added
    p8priority 8
    on Aug 7, 2026

  13. natechadwick-intsof commented on Aug 7, 2026

    @natechadwick-intsof
    natechadwick-intsof
    on Aug 7, 2026
    Collaborator

    Batch 1 (real generics) PR opened: #2322

    • Fixed ~50 rawtypes/unchecked/dep-ann diagnostics with real types (calendar + content-list helpers + @deprecated annotations)
    • cd modules/extensions-sfp && ../../mvnw.cmd clean install → BUILD SUCCESS; Tests run: 13, Failures: 0
    • Project default -Xlint: 0 javac warnings
    • Residual (~150 under full strip of remaining class suppressions): [8.2] extensions-sfp javac warnings residual after batch 1 (#2035) #2323
    • Keep this issue open until residual lands (do not close with Fixes yet)

    Parent tracker: #2200

    Co-Authored by Grok Build using grok-4.5 with agent main.

  14. natechadwick-intsof
    added a commit that references this issue on Aug 7, 2026

    fix(extensions-sfp): address review threads for batch 1 (#2035)

    b96c0f2
  15. natechadwick
    added a commit that references this issue on Aug 7, 2026

    fix(extensions-sfp): real generics javac warnings batch 1 (#2035) (#2322

    d17707b

  16. natechadwick-intsof commented on Aug 7, 2026

    @natechadwick-intsof
    natechadwick-intsof
    on Aug 7, 2026
    Collaborator

    Batch 2 residual (#2323) PR: #2345 — clears remaining class-level rawtypes/unchecked suppressions in site-folder cluster. PR includes Fixes #2035.

    Co-Authored by Grok Build using grok-4.5 with agent main.

  17. natechadwick
    closed this as completedin #2345on Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    8.2This issue found in 8.2 releasep8priority 8tech-debttech debt

    Type

    Task

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions