On Fri, 13 Sep 2024 09:03:55 GMT, Antón Seoane <[email protected]> wrote:
> Hi all, > > Currently, the Unified Logging framework defaults to three decorators > (uptime, level, tags) whenever the user does not specify otherwise through > `-Xlog`. This can result in cumbersome input whenever a specific user that > relies on a particular tag(s) has some predefined needs. For example, C2 > developers rarely need decorations, and having to manually specify this every > time results inconvenient. > > To address this, this PR enables the possibility of adding tag-specific > default decorators to UL. These defaults are in no way overriding user input > -- they will only act whenever `-Xlog` has no decorators supplied and there > is a positive match with the pre-specified defaults. Such a match is based on > the following: > > - Inclusion: if `-Xlog:jit+compilation` is provided, a default for `jit` may > be applied. > - Specificity: if, for the above line, there is a more specific default for > `jit+compilation` the latter shall be applied. Upon equal specificity cases, > both defaults will be applied. > - Additionally, defaults may target a specific log level. > > Decorators are also associated with an output file, so an output device may > only have one set of decorators. For this reason, if different > `LogSelection`s trigger defaults, none is to be applied. > > In summary, these defaults may be seen as a "tailored" or "flavoured" version > of the existing "uptime-level-tags" current defaults. > > Please consider this PR, and thanks! These defaults are not meant to target a specific selected output, so nothing different would occur. With respect to the first question, right now we would not get any defaults applied as there is a "collision" between A+B and C+D. That was my original idea, where I assumed it might be unwanted to apply all the possible defaults one over another. However, now (a) I don't think we will have that many defaults to drive this to chaos, and (b) as per @robcasloz feedback I believe it would be useful to apply all. Going back to your question: right now we do not apply any defaults upon that `-Xlog:A+B,C+D`, but I am working on some changes I will push soon that will change this behaviour to "merge" the default decorators for A+B and C+D ------------- PR Comment: https://git.openjdk.org/jdk/pull/20988#issuecomment-2370628280
