[GitHub] [logging-log4j2] carterkozak commented on issue #331: Fixed check for RollingFileManager, directWrite

2020-01-13 Thread GitBox
carterkozak commented on issue #331: Fixed check for RollingFileManager,directWrite URL: https://github.com/apache/logging-log4j2/pull/331#issuecomment-573773269 It looks like we have five occurrences of `instanceof DirectWriteRolloverStrategy` across `RollingFileManager` and `RollingRand

[GitHub] [logging-log4j2] ChristophKaser commented on issue #331: Fixed check for RollingFileManager, directWrite

2020-01-14 Thread GitBox
ChristophKaser commented on issue #331: Fixed check for RollingFileManager,directWrite URL: https://github.com/apache/logging-log4j2/pull/331#issuecomment-574099415 I only count three: - The one I replaced - [Two](https://github.com/apache/logging-log4j2/blob/231f669df3f16c03e35

[GitHub] [logging-log4j2] ChristophKaser opened a new pull request #332: LOG4J2-2760: Fixes RollingFileAppender with DirectWriteRolloverStrategy and HtmlLayout

2020-01-15 Thread GitBox
ChristophKaser opened a new pull request #332: LOG4J2-2760: Fixes RollingFileAppender with DirectWriteRolloverStrategy and HtmlLayout URL: https://github.com/apache/logging-log4j2/pull/332 This PR consist of a regression test showcasing the problem(s) and a fix. See https://issues.ap

[GitHub] [logging-log4j2] ChristophKaser opened a new pull request #333: Fixes travis build

2020-01-15 Thread GitBox
ChristophKaser opened a new pull request #333: Fixes travis build URL: https://github.com/apache/logging-log4j2/pull/333 by skipping the install phase and using openjdk8 This is an automated message from the Apache Git Service

[GitHub] [logging-log4j2] jvz commented on issue #333: Fixes travis build

2020-01-15 Thread GitBox
jvz commented on issue #333: Fixes travis build URL: https://github.com/apache/logging-log4j2/pull/333#issuecomment-57473 We need two JDKs available to compile right now (8 and 9+). Not sure if this would work. This is an

[GitHub] [logging-log4j2] ChristophKaser commented on issue #333: Fixes travis build

2020-01-15 Thread GitBox
ChristophKaser commented on issue #333: Fixes travis build URL: https://github.com/apache/logging-log4j2/pull/333#issuecomment-574732219 For some reason, it did for me (see https://travis-ci.org/ChristophKaser/logging-log4j2/builds/637479679). I guess openjdk11 is installed by default. --

[GitHub] [logging-log4j2] ChristophKaser commented on issue #333: Fixes travis build

2020-01-15 Thread GitBox
ChristophKaser commented on issue #333: Fixes travis build URL: https://github.com/apache/logging-log4j2/pull/333#issuecomment-574734303 That being said, it looks like the after_success script does not work. But I think this is an unrelated problem. ``` Exception in thread "main"

[GitHub] [logging-log4j2] jvz commented on issue #333: Fixes travis build

2020-01-15 Thread GitBox
jvz commented on issue #333: Fixes travis build URL: https://github.com/apache/logging-log4j2/pull/333#issuecomment-574760214 Oh that's great news! Let's see if this builds now. This is an automated message from the Apache Git

[GitHub] [logging-log4j2] jvz merged pull request #333: Fixes travis build

2020-01-15 Thread GitBox
jvz merged pull request #333: Fixes travis build URL: https://github.com/apache/logging-log4j2/pull/333 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [logging-log4j2] jvz commented on issue #333: Fixes travis build

2020-01-15 Thread GitBox
jvz commented on issue #333: Fixes travis build URL: https://github.com/apache/logging-log4j2/pull/333#issuecomment-574767850 Build passed. Thanks so much for figuring this out! This is an automated message from the Apache Git

[GitHub] [logging-log4j2] reebow opened a new pull request #334: (doc) fix typos

2020-01-23 Thread GitBox
reebow opened a new pull request #334: (doc) fix typos URL: https://github.com/apache/logging-log4j2/pull/334 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [logging-log4net] ErtugrulKra opened a new pull request #55: Feature/rest appender

2020-01-23 Thread GitBox
ErtugrulKra opened a new pull request #55: Feature/rest appender URL: https://github.com/apache/logging-log4net/pull/55 Hello, I created a rest service log4net appender for appending log events to rest api. This is an

[GitHub] [logging-log4net] ErtugrulKra closed pull request #55: Feature/rest appender

2020-01-27 Thread GitBox
ErtugrulKra closed pull request #55: Feature/rest appender URL: https://github.com/apache/logging-log4net/pull/55 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [logging-log4net] ErtugrulKra opened a new pull request #55: Feature/rest appender

2020-01-27 Thread GitBox
ErtugrulKra opened a new pull request #55: Feature/rest appender URL: https://github.com/apache/logging-log4net/pull/55 Hello, I created a rest service log4net appender for appending log events to rest api. This is an

[GitHub] [logging-log4j2] vy opened a new pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-28 Thread GitBox
vy opened a new pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335 This is the very first draft of my work on contributing [log4j2-logstash-layout](/vy/log4j2-logstash-layout) into Log4j core. Please note that this is s

[GitHub] [logging-log4j2] vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-28 Thread GitBox
vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372016555 ## File path: log4j-layout-jackson-json-template/src/main/resources/JsonLayout.json ## @@ -0,0 +1,27

[GitHub] [logging-log4j2] vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-28 Thread GitBox
vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372025989 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/template/l

[GitHub] [logging-log4j2] vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-28 Thread GitBox
vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372026508 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/template/l

[GitHub] [logging-log4j2] vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-28 Thread GitBox
vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372025989 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/template/l

[GitHub] [logging-log4j2] carterkozak commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-28 Thread GitBox
carterkozak commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372026814 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/t

[GitHub] [logging-log4j2] jvz commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-28 Thread GitBox
jvz commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372090887 ## File path: log4j-core/src/main/java/org/apache/logging/log4j/core/util/Throwables.java ## @@ -40,

[GitHub] [logging-log4j2] jvz commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-28 Thread GitBox
jvz commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372096836 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/template/

[GitHub] [logging-log4j2] jvz commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-28 Thread GitBox
jvz commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372096836 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/template/

[GitHub] [logging-log4j2] jvz commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-28 Thread GitBox
jvz commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-579506548 > I chose `JsonTemplateLayout` as the new module name, which clearly communicates the intent. I am not totally convinced with the package

[GitHub] [logging-log4j2] vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372341467 ## File path: log4j-layout-jackson-json-template/src/main/resources/JsonLayout.json ## @@ -0,0 +1,27

[GitHub] [logging-log4j2] vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372341591 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/template/l

[GitHub] [logging-log4j2] vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372341908 ## File path: log4j-core/src/main/java/org/apache/logging/log4j/core/util/Throwables.java ## @@ -40,1

[GitHub] [logging-log4j2] vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r372344182 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/template/l

[GitHub] [logging-log4j2] vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-579728012 > > I chose `JsonTemplateLayout` as the new module name, which clearly communicates the intent. I am not totally convinced with the packag

[GitHub] [logging-log4j2] vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-579835151 @remkop, please note the revamp in ed1dda4. The results on my machine are as follows: ``` $ uname -a Linux tahta 4.15.0-70-ge

[GitHub] [logging-log4j2] remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-579841760 I’ll take a look but it will take me some time. This is an automated

[GitHub] [logging-log4j2] jvz commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
jvz commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-579864331 To add a test jar as a dependency, add `test-jar` to the `` definition.

[GitHub] [logging-log4j2] jvz edited a comment on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
jvz edited a comment on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-579864331 To add a test jar as a dependency, add `test-jar` to the `` definition. See for example [log4j-core-its/pom.xml](https://github.

[GitHub] [logging-log4j2] vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-579866133 > To add a test jar as a dependency, add `test-jar` to the `` definition. See for example [log4j-core-its/pom.xml](https://github.com/apa

[GitHub] [logging-log4j2] MariusVolkhart opened a new pull request #336: LOG4J2-2768 Add log(String, ...) overloads to LogBuilder

2020-01-29 Thread GitBox
MariusVolkhart opened a new pull request #336: LOG4J2-2768 Add log(String, ...) overloads to LogBuilder URL: https://github.com/apache/logging-log4j2/pull/336 The overloaded methods behave similarly to log(String, Object[]), but reduce the need for an Object[] allocation in certain code pa

[GitHub] [logging-log4j2] remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580075777 Showing only the average in ns/operation: ``` NoOpsample 27754662 492.377 ± 6.634 ns/op Thr

[GitHub] [logging-log4j2] remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-29 Thread GitBox
remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580077235 If you wonder why the `MpmcArrayQueue` benchmark numbers are higher than simple allocation, I am not sure either. My first gues

[GitHub] [logging-log4j2] vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580189684 > The NoOp benchmark should be renamed Allocate Check. > I don't believe that `MpmcArrayQueue` is lock-free The defini

[GitHub] [logging-log4j2] vy edited a comment on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
vy edited a comment on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580189684 > The NoOp benchmark should be renamed Allocate Check. > I don't believe that `MpmcArrayQueue` is lock-free Th

[GitHub] [logging-log4j2] richardstartin commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
richardstartin commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580196090 To be clear, that’s just a hunch. I haven’t run the benchmarks let alone profiled them.

[GitHub] [logging-log4j2] vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580197240 And the following question still stands open: Given TLA results constitute the baseline, what is the pool implied maximum performance degr

[GitHub] [logging-log4j2] remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580205695 Well, what is the trade off? What benefits do we gain in return of this worse performance? --

[GitHub] [logging-log4j2] remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580215790 > I would be interested to know the marginal cost of G1 in this case. The most interesting metric is MPMC/G1 vs MPMC/other GC I

[GitHub] [logging-log4j2] richardstartin commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
richardstartin commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580229964 > I don’t think we’re measuring the cost of GC anywhere, are we? At this scale, there are so many confounding factors w

[GitHub] [logging-log4j2] vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580240386 Triggered by @richardstartin's observation, why don't we allow all 3 co-exist in the code base? After all, the call site doesn't care abou

[GitHub] [logging-log4j2] remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580261245 Not sure I agree... What is the trade-off? What do users gain when they select something other than ThreadLocal?

[GitHub] [logging-log4j2] vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
vy commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580267355 > Not sure I agree... What is the trade-off? What do users gain when they select something other than ThreadLocal? If you'd recall,

[GitHub] [logging-log4j2] remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-30 Thread GitBox
remkop commented on issue #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#issuecomment-580583586 @vy Understood. That makes sense to me. This is an automated message

[GitHub] [logging-log4cxx] stephen-webb opened a new pull request #14: Replace ant build with cmake

2020-01-30 Thread GitBox
stephen-webb opened a new pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [logging-log4j2] vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-31 Thread GitBox
vy commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r373474960 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/template/l

[GitHub] [logging-log4j2] jvz commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-31 Thread GitBox
jvz commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r373564119 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/template/

[GitHub] [logging-log4j2] jvz commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-01-31 Thread GitBox
jvz commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r373566976 ## File path: log4j-layout-jackson-json-template/src/main/java/org/apache/logging/log4j/jackson/json/template/

[GitHub] [logging-log4cxx] jvz commented on issue #14: Replace ant build with cmake

2020-01-31 Thread GitBox
jvz commented on issue #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#issuecomment-580809182 I ❤️ PRs that have huge negative line counts like this! I'm not super familiar with the tooling in use, though I'll try this out over the weekend.

[GitHub] [logging-log4cxx] jvz commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
jvz commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373572922 ## File path: pom.xml ## @@ -65,7 +65,7 @@ scm:git:http://gitbox.apache.org/repos/asf/logging-log4cxx.gi

[GitHub] [logging-log4cxx] jvz commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
jvz commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373573832 ## File path: pom.xml ## @@ -65,7 +65,7 @@ scm:git:http://gitbox.apache.org/repos/asf/logging-log4cxx.gi

[GitHub] [logging-log4cxx] jvz commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
jvz commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373577696 ## File path: src/site/apt/building/cmake.apt ## @@ -0,0 +1,75 @@ +~~ Licensed to the Apache Software Foundation (ASF) un

[GitHub] [logging-log4cxx] jvz commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
jvz commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373577204 ## File path: src/site/apt/building/cmake.apt ## @@ -0,0 +1,75 @@ +~~ Licensed to the Apache Software Foundation (ASF) un

[GitHub] [logging-log4cxx] jvz commented on issue #12: Add support for building and testing with cmake

2020-01-31 Thread GitBox
jvz commented on issue #12: Add support for building and testing with cmake URL: https://github.com/apache/logging-log4cxx/pull/12#issuecomment-580815431 Can anything from this be combined with #14? I love the simplification if possible rather than supporting multiple build systems. --

[GitHub] [logging-log4cxx] ams-tschoening commented on issue #12: Add support for building and testing with cmake

2020-01-31 Thread GitBox
ams-tschoening commented on issue #12: Add support for building and testing with cmake URL: https://github.com/apache/logging-log4cxx/pull/12#issuecomment-580837297 From my understanding this is CMAKE without removing anything, while #14 is CMAKE with removing at least ANT, because in the

[GitHub] [logging-log4j2] bkuker opened a new pull request #337: Update ConfigurationSource.java

2020-01-31 Thread GitBox
bkuker opened a new pull request #337: Update ConfigurationSource.java URL: https://github.com/apache/logging-log4j2/pull/337 I think this needs to store the data so that resetInputStream() works. The jmx function LoggerContextAdmin::getConfigText throws a NPE if you have previously call

[GitHub] [logging-log4j2] rgoers commented on issue #337: Update ConfigurationSource.java

2020-01-31 Thread GitBox
rgoers commented on issue #337: Update ConfigurationSource.java URL: https://github.com/apache/logging-log4j2/pull/337#issuecomment-580913709 Can you create a Jira issue for this. It would also be great to include a unit test that fails without this fix. ---

[GitHub] [logging-log4cxx] jvz commented on issue #12: Add support for building and testing with cmake

2020-01-31 Thread GitBox
jvz commented on issue #12: Add support for building and testing with cmake URL: https://github.com/apache/logging-log4cxx/pull/12#issuecomment-580930215 > While we are at it, it should be discussed if CMAKE should replace Autotools on Linux as well, leaving only the ANT/MAVEN-parts in plac

[GitHub] [logging-log4j2] bkuker commented on issue #337: Update ConfigurationSource.java

2020-01-31 Thread GitBox
bkuker commented on issue #337: Update ConfigurationSource.java URL: https://github.com/apache/logging-log4j2/pull/337#issuecomment-580948242 Thanks, yes I'll do that Monday! This is an automated message from the Apache Git Se

[GitHub] [logging-log4cxx] rm5248 commented on issue #12: Add support for building and testing with cmake

2020-01-31 Thread GitBox
rm5248 commented on issue #12: Add support for building and testing with cmake URL: https://github.com/apache/logging-log4cxx/pull/12#issuecomment-580961254 Why use Maven at all at that point? What I do for C++ projects is to just generate the site with Doxygen; it's possible to then make

[GitHub] [logging-log4cxx] rm5248 commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
rm5248 commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373735482 ## File path: CMakeLists.txt ## @@ -0,0 +1,52 @@ +cmake_minimum_required(VERSION 3.13) Review comment: Is it pos

[GitHub] [logging-log4cxx] rm5248 commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
rm5248 commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373735942 ## File path: CMakeLists.txt ## @@ -0,0 +1,52 @@ +cmake_minimum_required(VERSION 3.13) +include(src/cmake/projectVersi

[GitHub] [logging-log4cxx] rm5248 commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
rm5248 commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373736174 ## File path: CMakeLists.txt ## @@ -0,0 +1,52 @@ +cmake_minimum_required(VERSION 3.13) +include(src/cmake/projectVersi

[GitHub] [logging-log4cxx] stephen-webb commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
stephen-webb commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373737584 ## File path: CMakeLists.txt ## @@ -0,0 +1,52 @@ +cmake_minimum_required(VERSION 3.13) +include(src/cmake/proje

[GitHub] [logging-log4cxx] stephen-webb commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
stephen-webb commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373738203 ## File path: CMakeLists.txt ## @@ -0,0 +1,52 @@ +cmake_minimum_required(VERSION 3.13) Review comment: I

[GitHub] [logging-log4cxx] rm5248 commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
rm5248 commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373739795 ## File path: CMakeLists.txt ## @@ -0,0 +1,52 @@ +cmake_minimum_required(VERSION 3.13) +include(src/cmake/projectVersi

[GitHub] [logging-log4cxx] rm5248 commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
rm5248 commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373740646 ## File path: src/main/include/CMakeLists.txt ## @@ -0,0 +1,101 @@ +# Configure +if(WIN32) +add_custom_target(configur

[GitHub] [logging-log4cxx] rm5248 commented on issue #14: Replace ant build with cmake

2020-01-31 Thread GitBox
rm5248 commented on issue #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#issuecomment-580969003 Also: this doesn't install a pkgconfig file at all; that would be needed for other non-cmake buildsystems to work with.

[GitHub] [logging-log4cxx] stephen-webb commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
stephen-webb commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373754082 ## File path: pom.xml ## @@ -65,7 +65,7 @@ scm:git:http://gitbox.apache.org/repos/asf/logging-

[GitHub] [logging-log4cxx] stephen-webb commented on a change in pull request #14: Replace ant build with cmake

2020-01-31 Thread GitBox
stephen-webb commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r373754317 ## File path: CMakeLists.txt ## @@ -0,0 +1,52 @@ +cmake_minimum_required(VERSION 3.13) +include(src/cmake/proje

[GitHub] [logging-log4cxx] stephen-webb commented on issue #14: Replace ant build with cmake

2020-01-31 Thread GitBox
stephen-webb commented on issue #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#issuecomment-580994002 @rm5248 Can you tell me what I would need to change to install a pkgconfig file? -

[GitHub] [logging-log4cxx] ams-tschoening commented on issue #12: Add support for building and testing with cmake

2020-02-01 Thread GitBox
ams-tschoening commented on issue #12: Add support for building and testing with cmake URL: https://github.com/apache/logging-log4cxx/pull/12#issuecomment-581015095 > Why use Maven at all at that point? What I do for C++ projects is to just generate the site with Doxygen;[...] Maven

[GitHub] [logging-log4cxx] rm5248 commented on issue #14: Replace ant build with cmake

2020-02-01 Thread GitBox
rm5248 commented on issue #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#issuecomment-581035485 @stephen-webb You can configure the [.pc.in file](https://github.com/rm5248/logging-log4cxx/blob/a3fd5a53aefb297dd4526acd156e11335e484c37/CMakeLists.txt#

[GitHub] [logging-log4j-scala] jvz merged pull request #3: add support for scala 2.13.0 [LOG4J2-2668]

2020-02-01 Thread GitBox
jvz merged pull request #3: add support for scala 2.13.0 [LOG4J2-2668] URL: https://github.com/apache/logging-log4j-scala/pull/3 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [logging-log4j-scala] jvz closed pull request #2: [LOG4J2-2668] Add support for Scala 2.13.0

2020-02-01 Thread GitBox
jvz closed pull request #2: [LOG4J2-2668] Add support for Scala 2.13.0 URL: https://github.com/apache/logging-log4j-scala/pull/2 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [logging-log4j-scala] jvz commented on issue #2: [LOG4J2-2668] Add support for Scala 2.13.0

2020-02-01 Thread GitBox
jvz commented on issue #2: [LOG4J2-2668] Add support for Scala 2.13.0 URL: https://github.com/apache/logging-log4j-scala/pull/2#issuecomment-581073804 See #3 for updated version of this PR. This is an automated message from th

[GitHub] [logging-log4j2] turbanoff opened a new pull request #338: use 'final' modifier for fields where possible

2020-02-03 Thread GitBox
turbanoff opened a new pull request #338: use 'final' modifier for fields where possible URL: https://github.com/apache/logging-log4j2/pull/338 Also removed 2 unused fields which my IDEA highlights as possible `final`, but fields were unused (leftovers after refactoring): 1. `DefaultLog

[GitHub] [logging-log4j2] turbanoff commented on issue #338: use 'final' modifier for fields where possible

2020-02-03 Thread GitBox
turbanoff commented on issue #338: use 'final' modifier for fields where possible URL: https://github.com/apache/logging-log4j2/pull/338#issuecomment-581656719 I didn't touch fields in samples, tests and perf modules. Field `org.apache.logging.log4j.core.layout.AbstractStringLayout#chars

[GitHub] [logging-log4net] ErtugrulKra closed pull request #55: Feature/rest appender

2020-02-04 Thread GitBox
ErtugrulKra closed pull request #55: Feature/rest appender URL: https://github.com/apache/logging-log4net/pull/55 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [logging-log4net] ErtugrulKra opened a new pull request #56: Feature/LOG4NET-644

2020-02-04 Thread GitBox
ErtugrulKra opened a new pull request #56: Feature/LOG4NET-644 URL: https://github.com/apache/logging-log4net/pull/56 I've create a Rest Appender based on Ado.Net Appender. Unit tests and XML Samples are included the code repository. Newtonsoft JSON library added as dependency

[GitHub] [logging-log4cxx] stephen-webb commented on a change in pull request #14: Replace ant build with cmake

2020-02-06 Thread GitBox
stephen-webb commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r376210024 ## File path: src/site/apt/building/cmake.apt ## @@ -0,0 +1,75 @@ +~~ Licensed to the Apache Software Foundatio

[GitHub] [logging-log4cxx] stephen-webb commented on a change in pull request #14: Replace ant build with cmake

2020-02-06 Thread GitBox
stephen-webb commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r376211434 ## File path: src/site/apt/building/cmake.apt ## @@ -0,0 +1,75 @@ +~~ Licensed to the Apache Software Foundatio

[GitHub] [logging-log4cxx] jvz commented on a change in pull request #14: Replace ant build with cmake

2020-02-07 Thread GitBox
jvz commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r376486473 ## File path: src/site/apt/building/cmake.apt ## @@ -0,0 +1,75 @@ +~~ Licensed to the Apache Software Foundation (ASF) un

[GitHub] [logging-log4cxx] jvz commented on a change in pull request #14: Replace ant build with cmake

2020-02-07 Thread GitBox
jvz commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r376487230 ## File path: src/site/apt/building/cmake.apt ## @@ -0,0 +1,75 @@ +~~ Licensed to the Apache Software Foundation (ASF) un

[GitHub] [logging-log4j2] rgoers commented on a change in pull request #338: use 'final' modifier for fields where possible

2020-02-08 Thread GitBox
rgoers commented on a change in pull request #338: use 'final' modifier for fields where possible URL: https://github.com/apache/logging-log4j2/pull/338#discussion_r374413039 ## File path: log4j-api/src/main/java/org/apache/logging/log4j/message/ParameterizedMessage.java #

[GitHub] [logging-log4j2] rgoers commented on a change in pull request #338: use 'final' modifier for fields where possible

2020-02-08 Thread GitBox
rgoers commented on a change in pull request #338: use 'final' modifier for fields where possible URL: https://github.com/apache/logging-log4j2/pull/338#discussion_r374412481 ## File path: log4j-1.2-api/src/main/java/org/apache/log4j/spi/ThrowableInformation.java ## @@ -2

[GitHub] [logging-log4j2] rgoers commented on a change in pull request #338: use 'final' modifier for fields where possible

2020-02-08 Thread GitBox
rgoers commented on a change in pull request #338: use 'final' modifier for fields where possible URL: https://github.com/apache/logging-log4j2/pull/338#discussion_r374413266 ## File path: log4j-api/src/main/java/org/apache/logging/log4j/message/StringFormattedMessage.java ###

[GitHub] [logging-log4j2] rgoers commented on a change in pull request #338: use 'final' modifier for fields where possible

2020-02-08 Thread GitBox
rgoers commented on a change in pull request #338: use 'final' modifier for fields where possible URL: https://github.com/apache/logging-log4j2/pull/338#discussion_r374412811 ## File path: log4j-api/src/main/java/org/apache/logging/log4j/message/FormattedMessage.java ## @

[GitHub] [logging-log4j2] rgoers merged pull request #337: Log4j 2LOG4J2-2770 Fix NPE in JMX reconfiguration

2020-02-08 Thread GitBox
rgoers merged pull request #337: Log4j 2LOG4J2-2770 Fix NPE in JMX reconfiguration URL: https://github.com/apache/logging-log4j2/pull/337 This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [logging-log4j2] turbanoff commented on a change in pull request #338: use 'final' modifier for fields where possible

2020-02-08 Thread GitBox
turbanoff commented on a change in pull request #338: use 'final' modifier for fields where possible URL: https://github.com/apache/logging-log4j2/pull/338#discussion_r376738565 ## File path: log4j-1.2-api/src/main/java/org/apache/log4j/spi/ThrowableInformation.java ## @@

[GitHub] [logging-log4j2] turbanoff commented on a change in pull request #338: use 'final' modifier for fields where possible

2020-02-08 Thread GitBox
turbanoff commented on a change in pull request #338: use 'final' modifier for fields where possible URL: https://github.com/apache/logging-log4j2/pull/338#discussion_r376738565 ## File path: log4j-1.2-api/src/main/java/org/apache/log4j/spi/ThrowableInformation.java ## @@

[GitHub] [logging-log4j2] rgoers merged pull request #336: LOG4J2-2768 Add log(String, ...) overloads to LogBuilder

2020-02-08 Thread GitBox
rgoers merged pull request #336: LOG4J2-2768 Add log(String, ...) overloads to LogBuilder URL: https://github.com/apache/logging-log4j2/pull/336 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [logging-log4j2] rgoers commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout

2020-02-08 Thread GitBox
rgoers commented on a change in pull request #335: Import of LogstashLayout as JsonTemplateLayout URL: https://github.com/apache/logging-log4j2/pull/335#discussion_r376739348 ## File path: log4j-layout-jackson-json-template/src/main/resources/JsonLayout.json ## @@ -0,0 +1

[GitHub] [logging-log4j2] remkop commented on issue #338: use 'final' modifier for fields where possible

2020-02-08 Thread GitBox
remkop commented on issue #338: use 'final' modifier for fields where possible URL: https://github.com/apache/logging-log4j2/pull/338#issuecomment-583784498 As for why transient fields should not be `final`: https://stackoverflow.com/questions/2968876/final-transient-fields-and-serializatio

[GitHub] [logging-log4j2] rgoers merged pull request #334: (doc) fix typos

2020-02-08 Thread GitBox
rgoers merged pull request #334: (doc) fix typos URL: https://github.com/apache/logging-log4j2/pull/334 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [logging-log4cxx] stephen-webb commented on a change in pull request #14: Replace ant build with cmake

2020-02-08 Thread GitBox
stephen-webb commented on a change in pull request #14: Replace ant build with cmake URL: https://github.com/apache/logging-log4cxx/pull/14#discussion_r376748971 ## File path: src/site/apt/building/cmake.apt ## @@ -0,0 +1,75 @@ +~~ Licensed to the Apache Software Foundatio

<    21   22   23   24   25   26   27   28   29   30   >