LOG4J2-2785: Added support for pattern Layout to abbreviate logger or class names except n rightmost words

[spannm]

Hi, I've added a specialized abbreviator named DynamicWordAbbreviator to address the specific requirement of this feature request. Javadoc added and JUnit test updated to cover the new feature.

[carterkozak]

When I have some time to tinker, I think this feature could be added to the existing abbreviator. All existing patterns effectively use rightWordCount=1, however we could make that configurable and allow existing options to work with this.

Something along the lines of

int limit = input.length();
for (int i = 0; I < rightWordCount; I++) {
    limit = input.lastIndexOf('.', limit);
    if (limit < 0) {
        return input;
    }
}
// call into standard abbreviator.

[spannm]

[..] All existing patterns effectively use rightWordCount=1, however we could make that configurable and allow existing options to work with this.

@carterkozak I don't think this is the case. Consider patterns such as 1.1.1.*. The number of unmodified words at the right side depends on the length of the string to be abbreviated, not on the pattern. NameAbbreviator is abstract. Implementations are MaxElementAbbreviator and PatternAbbreviator. I don't see a good way of covering the feature request within these two implementations, and have therefore added DynamicWordAbbreviator.

[spannm]

@carterkozak Here's another small update to my pull request. Can you merge it into the release branch please?

[spannm]

"we love pull requests" they say you send them prs you listen to their criticism you incorporate their feedback you think you're done then nothing happens forever wtf

[carterkozak]

@sman-81 we are all volunteers and have limited time to devote to review. Your most recent comment comes across as entitled and rude. Please work on that. In your position, I might have replied on the PR with "Is there anything I can do to get this into a mergable state?"

you listen to their criticism you incorporate their feedback

I asked you twice to avoid string.split in favor of indexOf/lastIndexOf, but it's still used.

[spannm]

@carterkozak

I'm a volunteer just like you. My pull request is from July 1st, thus seven months old by now! The last comment from anyone other than me was August 12, 2021. Such discourages people from contributing to the project.

I asked you twice to avoid string.split in favor of indexOf/lastIndexOf, but it's still used.

And you are wrong. String#split does not create a Pattern object for certain one-character patterns such as here, as I have pointed out to you (see comment on August 5, 2021).

[carterkozak]

The point is that string.split is inefficient, regardless of internal details because it's allocating several new strings unnecessarily.

[spannm]

Hi there @carterkozak now String#split is gone. I have also squashed commits into one. Please take a look.

[rgoers]

This patch was manually extracted and merged since the backing repo has been deleted.

[spannm]

Hard to believe my PR actually got merged after a little over 1,5 years :)