[HBASE-26922] Fix LineLength warnings as much as possible if it can not be fixed by spotless

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.5.0, 3.0.0-alpha-3, 2.4.12
• Hadoop Flags: Reviewed
• Assignee: Duo Zhang
• Reporter: Duo Zhang
• Created: 02/Apr/22 05:45
• Updated: 10/Apr/22 20:48
• Resolved: 09/Apr/22 15:55

Description

One possible problem is the trailing comment, Like this

      assertTrue(initialRegions
        .contains(RegionReplicaUtil.getRegionInfoForReplica(mergedRegions.getFirst(), 1))); // this
                                                                                            // is
                                                                                            // the
                                                                                            // replica
                                                                                            // of
                                                                                            // the
                                                                                            // first
                                                                                            // region

Because of we limited line length, the format will be very strange and still has line length problem if some of the words are too long.

Let's move them from trailing to top of the statement.

Issue Links

• links to: GitHub Pull Request #4324