Because it isn’t a PatternConverter? Look at what it does. The newInstance method creates a DatePatternConverter. The FileDatePatternConverter and IntegerPatternConverter are both used for the filePattern in the rolling file appender, not for a pattern layout.
Ralph > On Aug 31, 2019, at 12:35 PM, Matt Sicker <[email protected]> wrote: > > FileDatePatternConverter, not DatePatternConverter. > > On Sat, 31 Aug 2019 at 14:35, Matt Sicker <[email protected]> wrote: >> >> Ah, this is an interesting historical design decision dating back to >> the very first commit! Why exactly does DatePatternConverter _not_ >> implement PatternConverter like literally every other converter class >> does? >> >> On Sat, 31 Aug 2019 at 14:29, Matt Sicker <[email protected]> wrote: >>> >>> Shit, I'm on it. >>> >>> On Sat, 31 Aug 2019 at 13:50, Apache Jenkins Server >>> <[email protected]> wrote: >>>> >>>> See >>>> <https://builds.apache.org/job/Log4j%202%203.x/476/display/redirect?page=changes> >>>> >>> >>> >>> -- >>> Matt Sicker <[email protected]> >> >> >> >> -- >> Matt Sicker <[email protected]> > > > > -- > Matt Sicker <[email protected]> >
