I have completed the work on custom levels.  It uses a variation of Nick’s 
“extensible enum” class.  The major difference with what he proposed is that 
the custom enums must be declared in a class annotated with @Plugin(name=“xxxx” 
category=“Level”) for them to be usable during configuration.

Are their any objections to me checking this in?  I’ll be doing the commit at 
around noon Pacific Daylight Time if I don’t hear any.

Ralph



On Jan 25, 2014, at 7:08 AM, Ralph Goers <ralph.go...@dslextreme.com> wrote:

> I am working on the implementation of custom levels now.  I should have it 
> done today.
> 
> Ralph
> 
> On Jan 24, 2014, at 7:07 PM, Remko Popma <remko.po...@gmail.com> wrote:
> 
>> What is the best way to make progress on the custom levels implementation?
>> 
>> Do we re-open LOG4J-41 or start a fresh Jira ticket? For implementation 
>> ideas, do we attach files to Jira, or create a branch?
>> 
>> Remko
>> 
>> On Saturday, January 25, 2014, Gary Gregory <garydgreg...@gmail.com> wrote:
>> On Fri, Jan 24, 2014 at 11:48 AM, Remko Popma <remko.po...@gmail.com> wrote:
>> Gary,
>> 
>> The hard-coded levels were proposed because it seemed that the extensible 
>> enum idea raised by Nick was not going to be accepted.
>> My original position was that Markers could fulfill the requirement but Nick 
>> and yourself made it clear that this was not satisfactory.
>> 
>> With extensible enums and markers off the table it seemed that the 
>> hard-coded levels was the only alternative, and discussion ensued about what 
>> these levels should be called and what strength they should have.
>> 
>> During this discussion, several people, including me, repeatedly expressed 
>> strong reservations about adding pre-defined levels, but by this time I 
>> think people were thinking there was no alternative.
>> 
>> It looked like we were getting stuck, with half the group moving in one 
>> direction ("add pre-defined levels!") and the other half wanting to move in 
>> another direction ("don't add pre-defined levels!"). I asked that we 
>> re-reviewed our assumptions and try to reach a solution that would satisfy 
>> all users. 
>> 
>> We then decided to explore the option of using extensible enums again. This 
>> is still ongoing, but I haven't seen anyone arguing against this idea since 
>> we started this thread.
>> 
>> Hard-coded levels and the extensible enum are different solutions to the 
>> same problem.
>> 
>> Hello All:
>> 
>> Absolutely not. See my DEFCON example. 
>> Talking about an "extensible enum" is mixing design and implementation, we 
>> are talking about 'custom' and/or 'extensible' levels.
>> Custom/Extensible levels can be designed to serve one or all of:
>> 
>> - Allow inserting custom levels between built-in levels.
>> - Allow for domain specific levels outside of the concept of built-in 
>> levels, the DEFCON example.
>> - Should the custom levels themselves be extensible?
>> 
>> Gary
>>  
>> The extensible enum solution satisfies all of us who are opposed to adding 
>> pre-defined levels, while also satisfying the original requirement raised by 
>> Nick and yourself. Frankly I don't understand why you would still want the 
>> pre-defined levels.
>> 
>> Remko
>> 
>> 
>> 
>> On Sat, Jan 25, 2014 at 12:53 AM, Gary Gregory <garydgreg...@gmail.com> 
>> wrote:
>> On Thu, Jan 23, 2014 at 10:45 PM, Remko Popma <remko.po...@gmail.com> wrote:
>> Gary, 
>> 
>> I think that's a very cool idea!
>> Much more flexible, powerful and elegant than pre-defined levels could ever 
>> be. 
>> 
>> As I wrote: "I am discussing custom levels here with the understanding that 
>> this is a separate topic from what the built-in levels are."
>> 
>> I'm not sure why you want to make the features mutually exclusive. (Some) 
>> others agree that these are different features.
>> 
>> I see two topics:
>> 
>> - What are the default levels for a 21st century logging framework. Do we 
>> simply blindly copy Log4j 1? Or do we look at frameworks from different 
>> languages and platforms for inspiration?
>> - How (not if, I think we all agree) should we allow for custom levels.
>> 
>> Gary
>> 
>> It definitely makes sense to design the extensible enum with this potential 
>> usage in mind. 
>> 
>> Remko
>> 
>> 
>> On Friday, January 24, 2014, Gary Gregory <garydgreg...@gmail.com> wrote:
>> I am discussing custom levels here with the understanding that this is a 
>> separate topic from what the built-in levels are. Here is how I convinced 
>> myself that custom levels are a “good thing”.
>> 
>> No matter which built-in levels exits, I may want custom levels. For 
>> example, I want my app to use the following levels DEFCON1, DEFCON2, 
>> DEFCON3, DEFCON4, and DEFCON5. This might be for one part of my app or a 
>> whole subsystem, no matter, I want to use the built-in levels in addition to 
>> the DEFCON levels. It is worth mentioning that if I want that feature only 
>> as a user, I can “skin” levels in a layout and assign any label to the 
>> built-in levels. If I am also a developer, I want to use DEFCON levels in 
>> the source code.
>> 
>> 
>> 
>> At first, my code might look like:
>> 
>> 
>> 
>> logger.log(DefconLevels.DEFCON5, “All is quiet”);
>> 
>> 
>> 
>> Let’s put aside for now the type of DefconLevels.DEFCON* objects. I am a 
>> user, and I care about my call sites.
>> 
>> 
>> 
>> What I really want of course is to write:
>> 
>> 
>> 
>> defconLogger.defcon5(“All is quiet”)
>> 
>> 
>> 
>> Therefore, I argue that for any “serious” use of a custom level, I will wrap 
>> a Logger in a custom logger class providing call-site friendly methods like 
>> defcon5(String).
>> 
>> 
>> 
>> So now, as a developer, all I care about is DefConLogger. It might wrap (or 
>> subclass) the Log4J Logger, who knows. The implementation of DefConLogger is 
>> not important to the developer (all I care is that the class has ‘defconN’ 
>> method) but it is important to the configuration author. This tells me that 
>> as a developer I do not care how DefConLogger is implemented, with custom 
>> levels, markers, or elves. However, as configuration author, I also want to 
>> use DEFCON level just like the built-in levels.
>> 
>> 
>> 
>> The configuration code co
>> 
>> 
>> 
>> 
>> -- 
>> E-Mail: garydgreg...@gmail.com | ggreg...@apache.org 
>> Java Persistence with Hibernate, Second Edition
>> JUnit in Action, Second Edition
>> Spring Batch in Action
>> Blog: http://garygregory.wordpress.com 
>> Home: http://garygregory.com/
>> Tweet! http://twitter.com/GaryGregory
> 

Reply via email to