Repository navigation
[8.2] Resolve javac warnings in perc-polls-services module #2042
Description
Description
The perc-polls-services 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
perc-polls-services
Build Summary
- Module:
perc-polls-services - Build Status: SUCCESS
- Reactor Index: 42
- Total Diagnostics: 4
- Javac Warnings: 4 (unique: 4)
- 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:allsettings. - 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 4 diagnostic message(s) are reported during compilation.
Acceptance Criteria
- Resolve all javac and javadoc warnings in the
perc-polls-servicesmodule. - 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
Activity
fix(perc-polls-services): suppress this-escape and serial warnings (#…
vijaya-boddipudi commentedon Aug 6, 2026
Status update (2026-08-06): The original PR for this issue (#2054) used
@SuppressWarningsannotations 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.natechadwick-intsof commentedon Aug 7, 2026
Agent progress: Claiming for overnight implement (night-issue-prs).
Scope: resolve 4 javac warnings in
perc-polls-services(deliverytiersuite/delivery-tier-suite/polls). Parent tracker: #2200.natechadwick-intsof commentedon Aug 7, 2026
PR opened: #2302
Real-fix replacement for suppress-only #2054:
PSPollsApplicationisfinal(this-escape)SerializablefromPSPoll/PSPollAnswer/IPSPollAnswer(serial lint without breaking HibernateSetmapping)Module
mvnw clean install: 8 tests, 0 failures, 0 javac warnings.