[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15298448#comment-15298448
]
Remko Popma commented on LOG4J2-623:
The answer to that was: no, just don't specify an
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15298299#comment-15298299
]
Remko Popma commented on LOG4J2-623:
Did you mean we should use these?
{code}
com.fast
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15298293#comment-15298293
]
Remko Popma commented on LOG4J2-623:
Can you clarify what you mean by the "default Jac
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15298283#comment-15298283
]
Gary Gregory commented on LOG4J2-623:
-
Also maybe the default Jackson behavior does wh
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15297672#comment-15297672
]
Remko Popma commented on LOG4J2-623:
*Quick summary of the problem*
XmlLayout, JsonLa
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15295468#comment-15295468
]
Mikael Ståldal commented on LOG4J2-623:
---
I think that we should consider at least th
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15295408#comment-15295408
]
Remko Popma commented on LOG4J2-623:
Just to double-check: this is not a showstopper f
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15295351#comment-15295351
]
Gary Gregory commented on LOG4J2-623:
-
Don't forget that there is YAML as well.
> Bet
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15295346#comment-15295346
]
Remko Popma commented on LOG4J2-623:
Since the same code is used for XML and JSON, who
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15274124#comment-15274124
]
Mikael Ståldal commented on LOG4J2-623:
---
This is still not fixed.
> Better structur
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15237223#comment-15237223
]
Matt Sicker commented on LOG4J2-623:
Agreed on fixing.
> Better structure of Thread C
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15236990#comment-15236990
]
Mikael Ståldal commented on LOG4J2-623:
---
Nice if we can finally fix this.
> Better
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14939525#comment-14939525
]
Mikael Ståldal commented on LOG4J2-623:
---
Should we close this as Won't Fix since cha
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14534081#comment-14534081
]
Mikael Ståldal commented on LOG4J2-623:
---
Still not fixed in 2.2.
> Better structure
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14000394#comment-14000394
]
Ralph Goers commented on LOG4J2-623:
Mikael, I would say that the Javadoc for the comp
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13999668#comment-13999668
]
Mikael Ståldal commented on LOG4J2-623:
---
But how is the poor Log4j user supposed to
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13999896#comment-13999896
]
Ralph Goers commented on LOG4J2-623:
Gary, I would not be in favor of making Jackson a
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13999867#comment-13999867
]
Joern Huxhorn commented on LOG4J2-623:
--
@Ralph Ah, I see. I wasn't aware that JSON co
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13999223#comment-13999223
]
Ralph Goers commented on LOG4J2-623:
While I agree in principal, what is awkward about
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13999824#comment-13999824
]
Gary Gregory commented on LOG4J2-623:
-
Another path would be to redo XML and JSON conf
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13999290#comment-13999290
]
Matt Sicker commented on LOG4J2-623:
You're just supposed to add jackson-databind to y
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996580#comment-13996580
]
Ralph Goers commented on LOG4J2-623:
You don't have to represent multiple inheritence.
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13997696#comment-13997696
]
Ralph Goers commented on LOG4J2-623:
You don't want to be modifying Log4j's pom.xml.
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13997687#comment-13997687
]
Mikael Ståldal commented on LOG4J2-623:
---
I tried to remove the true from jackson-cor
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996554#comment-13996554
]
Gary Gregory commented on LOG4J2-623:
-
@Jeorn: The above XML and JSON was produced usi
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13998320#comment-13998320
]
Joern Huxhorn commented on LOG4J2-623:
--
I suppose moving the XML- and JSON-layouts in
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13997697#comment-13997697
]
Mikael Ståldal commented on LOG4J2-623:
---
Now I get this problem instead, probably no
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13997677#comment-13997677
]
Mikael Ståldal commented on LOG4J2-623:
---
I use an XML configuration file, and JSONLa
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13997734#comment-13997734
]
Mikael Ståldal commented on LOG4J2-623:
---
Then you need to clearly document when thos
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13997548#comment-13997548
]
Gary Gregory commented on LOG4J2-623:
-
Hm, let's see {{JsonProcessingException}} is in
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13997380#comment-13997380
]
Mikael Ståldal commented on LOG4J2-623:
---
The latest code from SVN does not seem to w
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13997357#comment-13997357
]
Mikael Ståldal commented on LOG4J2-623:
---
I did not know that markers support multipl
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996793#comment-13996793
]
Joern Huxhorn commented on LOG4J2-623:
--
@Gary Ah, ok. The earlier [comment from
Mika
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996585#comment-13996585
]
Ralph Goers commented on LOG4J2-623:
The above should say "if you really want grandpar
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996853#comment-13996853
]
Gary Gregory commented on LOG4J2-623:
-
@Joern Huxhorn : Yeah, previous Log4j 1 and 2 i
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996256#comment-13996256
]
Joern Huxhorn commented on LOG4J2-623:
--
If you are interested: [this|http://pastebin.
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996593#comment-13996593
]
Ralph Goers commented on LOG4J2-623:
Gary,
You seem to want to name all your elements
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996214#comment-13996214
]
Mikael Ståldal commented on LOG4J2-623:
---
About JSON schema for the contextMap, can't
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996655#comment-13996655
]
Gary Gregory commented on LOG4J2-623:
-
I like this latest marker representation. It's
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996568#comment-13996568
]
Gary Gregory commented on LOG4J2-623:
-
@Mikael Ståldal : Your example does not cut it,
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996210#comment-13996210
]
Mikael Ståldal commented on LOG4J2-623:
---
marker now looks like this:
{code}
"marke
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996192#comment-13996192
]
Joern Huxhorn commented on LOG4J2-623:
--
I'd suggest to use something like
{code:xml
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13996195#comment-13996195
]
Mikael Ståldal commented on LOG4J2-623:
---
Why are you going to undo the improvement t
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995487#comment-13995487
]
Gary Gregory commented on LOG4J2-623:
-
As a reminder, the goal is to be able to ship a
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995225#comment-13995225
]
Gary Gregory commented on LOG4J2-623:
-
Simple: If it is not this way, it is *not* poss
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995287#comment-13995287
]
Ralph Goers commented on LOG4J2-623:
That is hardly surprising since the JSON spec doe
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995192#comment-13995192
]
Ralph Goers commented on LOG4J2-623:
Unless you can provide a reason that the contextM
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995168#comment-13995168
]
Gary Gregory commented on LOG4J2-623:
-
You should expect the following before 2.0 (ASA
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995115#comment-13995115
]
Mikael Ståldal commented on LOG4J2-623:
---
Can you give a concrete example of what the
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995116#comment-13995116
]
Ralph Goers commented on LOG4J2-623:
Gary, the correct syntax for a map in JSON is
{
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995105#comment-13995105
]
Gary Gregory commented on LOG4J2-623:
-
I am putting the finishing touches on redoing t
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995060#comment-13995060
]
Mikael Ståldal commented on LOG4J2-623:
---
The syntax of the generated JSON is invalid
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13985672#comment-13985672
]
Mikael Ståldal commented on LOG4J2-623:
---
Ah, yes then I agree.
> Better structure o
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13985656#comment-13985656
]
Ralph Goers commented on LOG4J2-623:
I didn't mean in terms of structure. I mean the n
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13985647#comment-13985647
]
Mikael Ståldal commented on LOG4J2-623:
---
Currently, it does mimic the XMLLayout. Thi
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13984415#comment-13984415
]
Ralph Goers commented on LOG4J2-623:
I would expect that the JSONLayout would mimic th
[
https://issues.apache.org/jira/browse/LOG4J2-623?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13984410#comment-13984410
]
Gary Gregory commented on LOG4J2-623:
-
I am reconsidering some details of the JSON lay
57 matches
Mail list logo
