Praveen7294 opened a new pull request, #24492:
URL: https://github.com/apache/pulsar/pull/24492
Fixes #23079
### Motivation
<!-- Explain here the context, and why you're making that change. What is
the problem you're trying to solve. -->
### Modifications
To improve code quality and maintainability, I resolved all existing
Checkstyle violations in the codebase. This ensures consistent coding standards
across modules, reduces CI noise, and creates a cleaner environment for
contributors. It also helps streamline future development and reviews by
eliminating style-related distractions.
<!-- Describe the modifications you've done. -->
### Verifying this change
Fixed all Checkstyle violations in the test directories of the Apache Pulsar
repository. The changes address issues related to `ImportOrder`, extra blank
lines between import statements, `LineLength`, `StaticVariableName`,
`ClassName`, `MethodName`, `FileTabCharacter`, `RegexpSingleline`, and other
style checks. This ensures cleaner, consistent, and CI-compliant test code
throughout the project.
Added the following specific suppressions because fixing the Checkstyle
violations would require changes to the existing code.
```
<suppress files="ExtensibleLoadManagerImplTest.java" checks="LineLength"/>
<suppress files="BundleSplitterTaskTest.java" checks="TodoComment"/>
<suppress files="BlobStoreTestBase.java" checks="TodoComment"/>
<suppress files="BlobStoreManagedLedgerOffloaderBase.java"
checks="TodoComment"/>
<suppress files="DebeziumMsSqlSourceTester.java"
checks="RegexpSinglelineJava"/>
<suppress files="DebeziumOracleDbSourceTester.java"
checks="RegexpSinglelineJava"/>
<suppress files="PulsarIODebeziumSourceRunner.java"
checks="RegexpSinglelineJava"/>
```
- [x] Make sure that the change passes the CI checks.
### Does this pull request potentially affect one of the following parts:
<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
*If the box was checked, please highlight the changes*
- [ ] Dependencies (add or upgrade a dependency)
- [ ] The public API
- [ ] The schema
- [ ] The default values of configurations
- [ ] The threading model
- [ ] The binary protocol
- [ ] The REST endpoints
- [ ] The admin CLI options
- [ ] The metrics
- [ ] Anything that affects deployment
### Documentation
<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
- [ ] `doc` <!-- Your PR contains doc changes. -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update
later -->
- [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->
### Matching PR in forked repository
PR in forked repository:
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
