[MINOR] Code Cleanup

[sjhajharia]

What

Code Cleanup

Changes

Some common changes include

• Replace the Arrays.asList() with Collections.singletonList() wherever possible
• Cleaning up some if-else blocks
• Replacing some instances of String.builder() with String operations

[chia7712]

@sjhajharia Thanks for this great contribution. However, we normally do such refactor one module by one module. For example: #15967, #15955, and #15877

The huge PR is un-reviewable and so it will be easy to make broken changes. Hence, could you please separate it into small PRs?

[sjhajharia]

Hey @chia7712
Thanks for the comment. I was also in a split brain regarding the same. I will close this one and create child PRs with one/two modules targetted at a time for an easier review.
Thanks!

[sjhajharia]

Hey @chia7712
I have created 5 sub-PRs to address the cleanup module-by-module.

• [MINOR] Code Cleanup (Clients Module) #16049: Clients Module
• [MINOR] Code Cleanup - Streams #16050: Streams Module
• [MINOR]: Code Cleanup - Metadata module #16065: Metadata Module
• [MINOR] Code Cleanup - Connect Module #16066: Connect Module
• [MINOR] : Code Cleanup - Misc modules #16067: All other modules (as the number of changes per module were very less)

Hope that folks can chime in and provide their feedbacks.
Good Day!