Log4j 2
  1. Log4j 2
  2. LOG4J2-417

Fix Event Level / LoggerConfig Level table at the architecture documentation page

    Details

      Description

      Hi, as I studied the "Event Level / LoggerConfig Level table" at the architecture documentation I saw that the event level ALL will be discarded. But the event level OFF would pass for further processing.
      Is this correct?

      My first assumption was that ALL would be processed by all LoggerConfig level instead of the level OFF.

      In the case my assumption is correct I would attach a patch to the issue which would correct also a small typo.

        Activity

        Hide
        Remko Popma added a comment -

        Fixed by Gary in revision 1529989.
        Please verify and close.

        Show
        Remko Popma added a comment - Fixed by Gary in revision 1529989. Please verify and close.
        Hide
        Ralph Goers added a comment -

        Gary, if the patch was applied why is this still open?

        Show
        Ralph Goers added a comment - Gary, if the patch was applied why is this still open?
        Show
        Gary Gregory added a comment - Applied https://issues.apache.org/jira/secure/attachment/12607211/architecture.xml.patch Thank you!
        Hide
        Gary Gregory added a comment -

        I think it depends on what you are doing. If all you are doing is fixing typos, then one diff file for the whole manual is OK by me.

        Show
        Gary Gregory added a comment - I think it depends on what you are doing. If all you are doing is fixing typos, then one diff file for the whole manual is OK by me.
        Hide
        Andreas Opitz added a comment - - edited

        Hi Gary, I attached the path file. But is just the deletion of one word hence you changed some parts that I have seen.

        One question off the topic. I reading at the moment the documentation step by step. Is it for you guys easier that I collect all points that I see and create one issue for them? Or one issue for each page?

        Show
        Andreas Opitz added a comment - - edited Hi Gary, I attached the path file. But is just the deletion of one word hence you changed some parts that I have seen. One question off the topic. I reading at the moment the documentation step by step. Is it for you guys easier that I collect all points that I see and create one issue for them? Or one issue for each page?
        Hide
        Gary Gregory added a comment -

        Thank you for the report. I fixed the table. Please provide a patch for your other fixes based on the latest from trunk. Thank you!

        Show
        Gary Gregory added a comment - Thank you for the report. I fixed the table. Please provide a patch for your other fixes based on the latest from trunk. Thank you!

          People

          • Assignee:
            Gary Gregory
            Reporter:
            Andreas Opitz
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 0.1h
              0.1h
              Remaining:
              Remaining Estimate - 0.1h
              0.1h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development