[GitHub] [logging-log4j2] KristjanESPERANTO edited a comment on pull request #642: Add Link to Release History

2021-12-26 Thread GitBox
KristjanESPERANTO edited a comment on pull request #642: URL: https://github.com/apache/logging-log4j2/pull/642#issuecomment-1001190408 Yes, that would address my point. The master branch is now the entry point for people who want to start dealing with the code. Since I can also see curren

[GitHub] [logging-log4cxx] rm5248 merged pull request #87: Imported and slightly cleaned up throughput tests

2021-12-26 Thread GitBox
rm5248 merged pull request #87: URL: https://github.com/apache/logging-log4cxx/pull/87 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notificatio

[GitHub] [logging-log4j2] rm5248 merged pull request #657: Api separation documentation

2021-12-26 Thread GitBox
rm5248 merged pull request #657: URL: https://github.com/apache/logging-log4j2/pull/657 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notificati

[GitHub] [logging-log4j2] dependabot[bot] commented on pull request #600: Bump byteBuddyVersion from 1.11.0 to 1.12.1

2021-12-26 Thread GitBox
dependabot[bot] commented on pull request #600: URL: https://github.com/apache/logging-log4j2/pull/600#issuecomment-1001235253 Looks like these dependencies are no longer a dependency, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [logging-log4j2] dependabot[bot] closed pull request #600: Bump byteBuddyVersion from 1.11.0 to 1.12.1

2021-12-26 Thread GitBox
dependabot[bot] closed pull request #600: URL: https://github.com/apache/logging-log4j2/pull/600 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: n

[GitHub] [logging-log4j2] dependabot[bot] commented on pull request #606: Bump spotbugs from 4.1.2 to 4.5.0

2021-12-26 Thread GitBox
dependabot[bot] commented on pull request #606: URL: https://github.com/apache/logging-log4j2/pull/606#issuecomment-1001236731 Looks like com.github.spotbugs:spotbugs is no longer a dependency, so this is no longer needed. -- This is an automated message from the Apache Git Service. To r

[GitHub] [logging-log4j2] dependabot[bot] closed pull request #606: Bump spotbugs from 4.1.2 to 4.5.0

2021-12-26 Thread GitBox
dependabot[bot] closed pull request #606: URL: https://github.com/apache/logging-log4j2/pull/606 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: n

[GitHub] [logging-log4j2] dependabot[bot] commented on pull request #408: Bump maven-shade-plugin from 2.2 to 3.2.4

2021-12-26 Thread GitBox
dependabot[bot] commented on pull request #408: URL: https://github.com/apache/logging-log4j2/pull/408#issuecomment-1001237113 Looks like org.apache.maven.plugins:maven-shade-plugin is up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Servic

[GitHub] [logging-log4j2] dependabot[bot] closed pull request #408: Bump maven-shade-plugin from 2.2 to 3.2.4

2021-12-26 Thread GitBox
dependabot[bot] closed pull request #408: URL: https://github.com/apache/logging-log4j2/pull/408 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: n

[GitHub] [logging-log4j2] dependabot[bot] closed pull request #494: Bump spring-ws-core from 2.1.4.RELEASE to 3.1.1

2021-12-26 Thread GitBox
dependabot[bot] closed pull request #494: URL: https://github.com/apache/logging-log4j2/pull/494 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: n

[GitHub] [logging-log4j2] dependabot[bot] commented on pull request #494: Bump spring-ws-core from 2.1.4.RELEASE to 3.1.1

2021-12-26 Thread GitBox
dependabot[bot] commented on pull request #494: URL: https://github.com/apache/logging-log4j2/pull/494#issuecomment-1001237745 Superseded by #661. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [logging-log4j2] dependabot[bot] opened a new pull request #661: Bump spring-ws-core from 2.1.4.RELEASE to 3.1.2

2021-12-26 Thread GitBox
dependabot[bot] opened a new pull request #661: URL: https://github.com/apache/logging-log4j2/pull/661 Bumps [spring-ws-core](https://github.com/spring-projects/spring-ws) from 2.1.4.RELEASE to 3.1.2. Commits https://github.com/spring-projects/spring-ws/commit/7c7e86bfa54606a7

[GitHub] [logging-log4j2] dependabot[bot] opened a new pull request #662: Bump tomcat-catalina from 8.5.20 to 10.0.14

2021-12-26 Thread GitBox
dependabot[bot] opened a new pull request #662: URL: https://github.com/apache/logging-log4j2/pull/662 Bumps tomcat-catalina from 8.5.20 to 10.0.14. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.apache.to

[GitHub] [logging-log4j2] dependabot[bot] commented on pull request #581: Bump tomcat-catalina from 8.5.20 to 10.0.11

2021-12-26 Thread GitBox
dependabot[bot] commented on pull request #581: URL: https://github.com/apache/logging-log4j2/pull/581#issuecomment-1001237818 Superseded by #662. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [logging-log4j2] dependabot[bot] closed pull request #581: Bump tomcat-catalina from 8.5.20 to 10.0.11

2021-12-26 Thread GitBox
dependabot[bot] closed pull request #581: URL: https://github.com/apache/logging-log4j2/pull/581 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: n

[GitHub] [logging-log4j2] dependabot[bot] opened a new pull request #663: Bump awaitility from 4.0.3 to 4.1.1

2021-12-26 Thread GitBox
dependabot[bot] opened a new pull request #663: URL: https://github.com/apache/logging-log4j2/pull/663 Bumps [awaitility](https://github.com/awaitility/awaitility) from 4.0.3 to 4.1.1. Changelog Sourced from https://github.com/awaitility/awaitility/blob/master/changelog.txt";>awai

[GitHub] [logging-log4j2] dependabot[bot] opened a new pull request #664: Bump liquibase-core from 3.5.5 to 4.6.2

2021-12-26 Thread GitBox
dependabot[bot] opened a new pull request #664: URL: https://github.com/apache/logging-log4j2/pull/664 Bumps [liquibase-core](https://github.com/liquibase/liquibase) from 3.5.5 to 4.6.2. Release notes Sourced from https://github.com/liquibase/liquibase/releases";>liquibase-core's

[GitHub] [logging-log4j2] dependabot[bot] commented on pull request #603: Bump spring-boot.version from 2.5.7 to 2.6.0

2021-12-26 Thread GitBox
dependabot[bot] commented on pull request #603: URL: https://github.com/apache/logging-log4j2/pull/603#issuecomment-1001239295 Superseded by #665. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [logging-log4j2] dependabot[bot] opened a new pull request #665: Bump spring-boot.version from 2.5.7 to 2.6.2

2021-12-26 Thread GitBox
dependabot[bot] opened a new pull request #665: URL: https://github.com/apache/logging-log4j2/pull/665 Bumps `spring-boot.version` from 2.5.7 to 2.6.2. Updates `spring-boot-dependencies` from 2.5.7 to 2.6.2 Release notes Sourced from https://github.com/spring-projects/spring-boo

[GitHub] [logging-log4j2] dependabot[bot] closed pull request #603: Bump spring-boot.version from 2.5.7 to 2.6.0

2021-12-26 Thread GitBox
dependabot[bot] closed pull request #603: URL: https://github.com/apache/logging-log4j2/pull/603 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: n

[GitHub] [logging-log4j2] dependabot[bot] opened a new pull request #666: Bump org.apache.felix.framework from 5.6.12 to 7.0.3

2021-12-26 Thread GitBox
dependabot[bot] opened a new pull request #666: URL: https://github.com/apache/logging-log4j2/pull/666 Bumps org.apache.felix.framework from 5.6.12 to 7.0.3. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.

[GitHub] [logging-log4j2] carterkozak opened a new pull request #667: Fix ExtendedLoggerWrapper.logMessage double-logging

2021-12-26 Thread GitBox
carterkozak opened a new pull request #667: URL: https://github.com/apache/logging-log4j2/pull/667 Previously when the logger instance was location-aware and the logger was configured to require location info, both branches were invoked rather than only the location-aware branch. --

[GitHub] [logging-log4cxx] swebb2066 opened a new pull request #88: Improve throughput when using disabled logging statements

2021-12-26 Thread GitBox
swebb2066 opened a new pull request #88: URL: https://github.com/apache/logging-log4cxx/pull/88 Change the pointer to the Hierarchy from a std::weak_ptr to a raw pointer to avoid the lock(0 in Logger::isDisabled() functions. Also disconnects the Logger from the Hierarchy when the Hie

[GitHub] [logging-log4cxx] swebb2066 opened a new pull request #89: Prevent thoughputtests warning

2021-12-26 Thread GitBox
swebb2066 opened a new pull request #89: URL: https://github.com/apache/logging-log4cxx/pull/89 This PR is required to prevent 'No appender could be found for logger (bench_logger)' warning in throughputtests after applying #88 -- This is an automated message from the Apache Git Service

[GitHub] [logging-log4cxx] swebb2066 commented on pull request #73: Throughput tests

2021-12-26 Thread GitBox
swebb2066 commented on pull request #73: URL: https://github.com/apache/logging-log4cxx/pull/73#issuecomment-1001350144 This has been superceded by #87 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [logging-log4cxx] swebb2066 closed pull request #73: Throughput tests

2021-12-26 Thread GitBox
swebb2066 closed pull request #73: URL: https://github.com/apache/logging-log4cxx/pull/73 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifica

[GitHub] [logging-log4j2] Kenducky opened a new pull request #668: update log4j-config.xsd fixed one default type

2021-12-26 Thread GitBox
Kenducky opened a new pull request #668: URL: https://github.com/apache/logging-log4j2/pull/668 some PatternLayOut.java source code about "disableAnsi" in master branch like this: If {@code "true"} (default is value of system property `log4j.skipJansi`, or `true` if undefined), do not o

[GitHub] [logging-log4cxx] swebb2066 opened a new pull request #90: [LOGCXX-546] Prevent unnecessary locking when determining the logging level

2021-12-26 Thread GitBox
swebb2066 opened a new pull request #90: URL: https://github.com/apache/logging-log4cxx/pull/90 **throughputtests after these changes:** ``` Benchmarking library only(no writing out): ** Benchmarking Single threaded: 100

[GitHub] [logging-log4cxx] swebb2066 closed pull request #89: Prevent thoughputtests warning

2021-12-26 Thread GitBox
swebb2066 closed pull request #89: URL: https://github.com/apache/logging-log4cxx/pull/89 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifica

[GitHub] [logging-log4cxx] swebb2066 commented on pull request #89: Prevent thoughputtests warning

2021-12-26 Thread GitBox
swebb2066 commented on pull request #89: URL: https://github.com/apache/logging-log4cxx/pull/89#issuecomment-1001372552 Suerceded by #90 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the speci

[GitHub] [logging-log4cxx] swebb2066 commented on pull request #88: Improve throughput when using disabled logging statements

2021-12-26 Thread GitBox
swebb2066 commented on pull request #88: URL: https://github.com/apache/logging-log4cxx/pull/88#issuecomment-1001372699 Superceded by #90 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the spec

[GitHub] [logging-log4j2] garydgregory commented on pull request #667: Fix ExtendedLoggerWrapper.logMessage double-logging

2021-12-27 Thread GitBox
garydgregory commented on pull request #667: URL: https://github.com/apache/logging-log4j2/pull/667#issuecomment-1001585742 Yikes, nice find @carterkozak 👍 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [logging-log4j2] vorburger commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
vorburger commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775528603 ## File path: pom.xml ## @@ -964,6 +964,19 @@ 0.8.1 test + +com.google.guava + +guava-testl

[GitHub] [logging-log4j2] vorburger commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
vorburger commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775572900 ## File path: log4j-to-jul/src/main/java/org/apache/logging/log4j/tojul/JULLogger.java ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [logging-log4j2] vorburger commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
vorburger commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775574350 ## File path: log4j-to-jul/src/main/java/org/apache/logging/log4j/tojul/JULLogger.java ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [logging-log4j2] vorburger commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
vorburger commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775575152 ## File path: log4j-to-jul/src/main/java/org/apache/logging/log4j/tojul/JULLogger.java ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [logging-log4j2] vorburger commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
vorburger commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775577767 ## File path: log4j-to-jul/src/main/java/org/apache/logging/log4j/tojul/JULLogger.java ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [logging-log4j2] ramananravi commented on pull request #670: add support for FileSize in TB

2021-12-27 Thread GitBox
ramananravi commented on pull request #670: URL: https://github.com/apache/logging-log4j2/pull/670#issuecomment-1001669932 Currently the default value of 'exceeds' attribute of 'IfAccumulatedFileSize' is Long.MAX_VALUE, which is considerably larger than 1000 TB itself. Hence adding support

[GitHub] [logging-log4j2] carterkozak merged pull request #667: LOG4J2-3292 Fix ExtendedLoggerWrapper.logMessage double-logging

2021-12-27 Thread GitBox
carterkozak merged pull request #667: URL: https://github.com/apache/logging-log4j2/pull/667 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notif

[GitHub] [logging-log4j2] ramananravi closed pull request #670: add support for FileSize in TB

2021-12-27 Thread GitBox
ramananravi closed pull request #670: URL: https://github.com/apache/logging-log4j2/pull/670 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notif

[GitHub] [logging-log4j2] carterkozak commented on a change in pull request #644: LOG4J2-3259: Limit max recursion depth when interpolating strings.

2021-12-27 Thread GitBox
carterkozak commented on a change in pull request #644: URL: https://github.com/apache/logging-log4j2/pull/644#discussion_r775600190 ## File path: log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/StrSubstitutor.java ## @@ -986,12 +986,20 @@ protected boolean subst

[GitHub] [logging-log4j2] carterkozak commented on a change in pull request #644: LOG4J2-3259: Limit max recursion depth when interpolating strings.

2021-12-27 Thread GitBox
carterkozak commented on a change in pull request #644: URL: https://github.com/apache/logging-log4j2/pull/644#discussion_r775600190 ## File path: log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/StrSubstitutor.java ## @@ -986,12 +986,20 @@ protected boolean subst

[GitHub] [logging-log4j2] carterkozak commented on a change in pull request #644: LOG4J2-3259: Limit max recursion depth when interpolating strings.

2021-12-27 Thread GitBox
carterkozak commented on a change in pull request #644: URL: https://github.com/apache/logging-log4j2/pull/644#discussion_r775600190 ## File path: log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/StrSubstitutor.java ## @@ -986,12 +986,20 @@ protected boolean subst

[GitHub] [logging-log4j2] carterkozak commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
carterkozak commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775604217 ## File path: pom.xml ## @@ -964,6 +964,19 @@ 0.8.1 test + +com.google.guava + +guava-tes

[GitHub] [logging-log4j2] carterkozak commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
carterkozak commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775604539 ## File path: log4j-to-jul/src/main/java/org/apache/logging/log4j/tojul/JULLogger.java ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Softw

[GitHub] [logging-log4j2] ramananravi commented on pull request #671: add support for FileSize in TB

2021-12-27 Thread GitBox
ramananravi commented on pull request #671: URL: https://github.com/apache/logging-log4j2/pull/671#issuecomment-1001690144 Currently the default value of 'exceeds' attribute of 'IfAccumulatedFileSize' is Long.MAX_VALUE, which is considerably larger than 1000 TB itself. Hence adding support

[GitHub] [logging-log4j2] vorburger commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
vorburger commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775607662 ## File path: pom.xml ## @@ -964,6 +964,19 @@ 0.8.1 test + +com.google.guava + +guava-testl

[GitHub] [logging-log4j2] carterkozak commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
carterkozak commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775607939 ## File path: log4j-to-jul/src/main/java/org/apache/logging/log4j/tojul/JULLogger.java ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Softw

[GitHub] [logging-log4j2] vorburger commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
vorburger commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775609043 ## File path: pom.xml ## @@ -964,6 +964,19 @@ 0.8.1 test + +com.google.guava + +guava-testl

[GitHub] [logging-log4j2] vorburger commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
vorburger commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775609433 ## File path: log4j-to-jul/src/main/java/org/apache/logging/log4j/tojul/JULLogger.java ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [logging-log4j2] vorburger commented on a change in pull request #653: (DRAFT, WIP - DO NOT MERGE YET, BUT INITIAL REVIEW WELCOME) Release 2.x log4j-to-jul JDK Logging Bridge (LOG4J2-3282)

2021-12-27 Thread GitBox
vorburger commented on a change in pull request #653: URL: https://github.com/apache/logging-log4j2/pull/653#discussion_r775609860 ## File path: log4j-to-jul/src/main/java/org/apache/logging/log4j/tojul/JULLogger.java ## @@ -0,0 +1,246 @@ +/* + * Licensed to the Apache Softwar

[GitHub] [logging-log4cxx] swebb2066 opened a new pull request #91: Allow throughputtests to be built when LOG4CXX_LOGCHAR_IS_WCHAR is on

2021-12-27 Thread GitBox
swebb2066 opened a new pull request #91: URL: https://github.com/apache/logging-log4cxx/pull/91 This PR completes #87 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To un

[GitHub] [logging-log4cxx] swebb2066 closed pull request #91: Allow throughputtests to be built when LOG4CXX_LOGCHAR_IS_WCHAR is on

2021-12-27 Thread GitBox
swebb2066 closed pull request #91: URL: https://github.com/apache/logging-log4cxx/pull/91 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifica

[GitHub] [logging-log4cxx] swebb2066 commented on pull request #91: Allow throughputtests to be built when LOG4CXX_LOGCHAR_IS_WCHAR is on

2021-12-27 Thread GitBox
swebb2066 commented on pull request #91: URL: https://github.com/apache/logging-log4cxx/pull/91#issuecomment-1001858047 Duplicates #90 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

[GitHub] [logging-log4cxx] swebb2066 opened a new pull request #92: Allow throughputtests to be built when LOGCHAR_IS_WCHAR is on

2021-12-27 Thread GitBox
swebb2066 opened a new pull request #92: URL: https://github.com/apache/logging-log4cxx/pull/92 This change adorns literal strings within LOG4CXX_STR where required. -- 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] vy commented on pull request #672: Excluded markdown from apache-rat check

2021-12-28 Thread GitBox
vy commented on pull request #672: URL: https://github.com/apache/logging-log4j2/pull/672#issuecomment-1002281324 @carterkozak, @jvz, why do we exclude docs? I think we just need to add license headers to them as any other file. (For the records, I've addressed this in `release-2.x` withou

[GitHub] [logging-log4j2] vy commented on pull request #671: add support for FileSize in TB

2021-12-28 Thread GitBox
vy commented on pull request #671: URL: https://github.com/apache/logging-log4j2/pull/671#issuecomment-1002287774 @ramananravi, thanks so much for the contribution. This looks like a legit request to me. Though I find the existing `(long) (double * long)` multiplications in `FileSize` dang

[GitHub] [logging-log4j2] jvz commented on pull request #672: Excluded markdown from apache-rat check

2021-12-28 Thread GitBox
jvz commented on pull request #672: URL: https://github.com/apache/logging-log4j2/pull/672#issuecomment-1002289256 Good catch. I think the commented out license header for markdown files makes sense. We can revert the rat exclusion. -- This is an automated message from the Apache Git Ser

[GitHub] [logging-log4j2] vy commented on pull request #669: fix: remove invalid badges

2021-12-28 Thread GitBox
vy commented on pull request #669: URL: https://github.com/apache/logging-log4j2/pull/669#issuecomment-1002290247 It wasn't invalid, but broken; fixed. @Xunzhuo, thanks so much for the heads up! -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [logging-log4j2] vy closed pull request #669: fix: remove invalid badges

2021-12-28 Thread GitBox
vy closed pull request #669: URL: https://github.com/apache/logging-log4j2/pull/669 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-

[GitHub] [logging-log4cxx] swebb2066 opened a new pull request #93: Add a unit test for the automatic configuration feature

2021-12-28 Thread GitBox
swebb2066 opened a new pull request #93: URL: https://github.com/apache/logging-log4cxx/pull/93 This new unit test covers the undocumented automatic configuration function. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [logging-log4cxx] rm5248 commented on pull request #90: [LOGCXX-546] Prevent unnecessary locking when determining the logging level

2021-12-28 Thread GitBox
rm5248 commented on pull request #90: URL: https://github.com/apache/logging-log4cxx/pull/90#issuecomment-1002365783 Looks good to me. I also get rather dramatic increases in the number of log messages/second when logging is disabled(approx. 10x on my machine). Other stats are about 5-10

[GitHub] [logging-log4cxx] swebb2066 commented on pull request #90: [LOGCXX-546] Prevent unnecessary locking when determining the logging level

2021-12-28 Thread GitBox
swebb2066 commented on pull request #90: URL: https://github.com/apache/logging-log4cxx/pull/90#issuecomment-1002412720 Do you think this performance fix warrent a release (0.12.3) before next_stable is ready? -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [logging-log4cxx] swebb2066 edited a comment on pull request #90: [LOGCXX-546] Prevent unnecessary locking when determining the logging level

2021-12-28 Thread GitBox
swebb2066 edited a comment on pull request #90: URL: https://github.com/apache/logging-log4cxx/pull/90#issuecomment-1002412720 Do you think this performance fix warrants a release (0.12.3) before next_stable is ready? -- This is an automated message from the Apache Git Service. To respon

[GitHub] [logging-log4cxx] ams-tschoening commented on a change in pull request #92: Allow throughputtests to be built when LOGCHAR_IS_WCHAR is on

2021-12-29 Thread GitBox
ams-tschoening commented on a change in pull request #92: URL: https://github.com/apache/logging-log4cxx/pull/92#discussion_r776230335 ## File path: src/test/cpp/throughput/throughput-main.cpp ## @@ -71,14 +74,14 @@ static void benchmark_conversion_pattern( std::string name,

[GitHub] [logging-log4cxx] rm5248 commented on pull request #90: [LOGCXX-546] Prevent unnecessary locking when determining the logging level

2021-12-29 Thread GitBox
rm5248 commented on pull request #90: URL: https://github.com/apache/logging-log4cxx/pull/90#issuecomment-1002651075 My current thought is that we'll do a 0.13.0 release in a few weeks; there are a number of fixes for that already in master. next_stable probably won't be for several month

[GitHub] [logging-log4j2] garydgregory commented on pull request #668: update log4j-config.xsd fixed one default type

2021-12-29 Thread GitBox
garydgregory commented on pull request #668: URL: https://github.com/apache/logging-log4j2/pull/668#issuecomment-1002652178 Hi @Kenducky Thank you for your PR. You'll want to look at the branch 'release-2.x', that where the 2.x releases originate while the branch 'master' is for futu

[GitHub] [logging-log4j2] sullis commented on pull request #446: mockito 3.6.28

2021-12-29 Thread GitBox
sullis commented on pull request #446: URL: https://github.com/apache/logging-log4j2/pull/446#issuecomment-1002749324 obsolete. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

[GitHub] [logging-log4j2] sullis closed pull request #446: mockito 3.6.28

2021-12-29 Thread GitBox
sullis closed pull request #446: URL: https://github.com/apache/logging-log4j2/pull/446 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notificati

[GitHub] [logging-log4j2] sullis commented on pull request #445: slf4j 1.7.30

2021-12-29 Thread GitBox
sullis commented on pull request #445: URL: https://github.com/apache/logging-log4j2/pull/445#issuecomment-1002749381 obsolete. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

[GitHub] [logging-log4j2] sullis closed pull request #445: slf4j 1.7.30

2021-12-29 Thread GitBox
sullis closed pull request #445: URL: https://github.com/apache/logging-log4j2/pull/445 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notificati

[GitHub] [logging-log4j2] sullis opened a new pull request #673: mockito 4.2.0

2021-12-29 Thread GitBox
sullis opened a new pull request #673: URL: https://github.com/apache/logging-log4j2/pull/673 latest version of Mockito -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

[GitHub] [logging-log4j2] sullis commented on pull request #673: mockito 4.2.0

2021-12-29 Thread GitBox
sullis commented on pull request #673: URL: https://github.com/apache/logging-log4j2/pull/673#issuecomment-1002759421 Mistaken PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific co

[GitHub] [logging-log4j2] sullis closed pull request #673: mockito 4.2.0

2021-12-29 Thread GitBox
sullis closed pull request #673: URL: https://github.com/apache/logging-log4j2/pull/673 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notificati

[GitHub] [logging-log4j2] carterkozak closed pull request #672: Excluded markdown from apache-rat check

2021-12-29 Thread GitBox
carterkozak closed pull request #672: URL: https://github.com/apache/logging-log4j2/pull/672 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notif

[GitHub] [logging-log4j2] carterkozak commented on pull request #672: Excluded markdown from apache-rat check

2021-12-29 Thread GitBox
carterkozak commented on pull request #672: URL: https://github.com/apache/logging-log4j2/pull/672#issuecomment-1002808751 This was meant to match the `README.md` file. No strong preference either way from my end, I opted for exclusion initially since it's easier to read through the file i

[GitHub] [logging-log4j2] vy commented on pull request #671: add support for FileSize in TB

2021-12-29 Thread GitBox
vy commented on pull request #671: URL: https://github.com/apache/logging-log4j2/pull/671#issuecomment-1002900021 @ramananravi, we think there are a few more places in the code where a similar improvement might be handy. Mind creating a JIRA ticket for this change, please? -- This is an

[GitHub] [logging-log4cxx] ams-tschoening merged pull request #92: Allow throughputtests to be built when LOGCHAR_IS_WCHAR is on

2021-12-29 Thread GitBox
ams-tschoening merged pull request #92: URL: https://github.com/apache/logging-log4cxx/pull/92 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: not

[GitHub] [logging-log4cxx] ams-tschoening commented on pull request #90: [LOGCXX-546] Prevent unnecessary locking when determining the logging level

2021-12-30 Thread GitBox
ams-tschoening commented on pull request #90: URL: https://github.com/apache/logging-log4cxx/pull/90#issuecomment-1002920713 Before: ```text Benchmarking library only(no writing out): ** Benchmarking Single threaded:

[GitHub] [logging-log4j2] RogerioBlanco opened a new pull request #675: Fix at start up the file index what should be

2021-12-30 Thread GitBox
RogerioBlanco opened a new pull request #675: URL: https://github.com/apache/logging-log4j2/pull/675 The DirectWriteRolloverStrategy should at the start up choose the last index if has eligible files instead the eligible files size. -- This is an automated message from the Apache Git Ser

[GitHub] [logging-log4j2] ramananravi commented on pull request #671: add support for FileSize in TB

2021-12-30 Thread GitBox
ramananravi commented on pull request #671: URL: https://github.com/apache/logging-log4j2/pull/671#issuecomment-1003021312 > @ramananravi, we think there are a few more places in the code where a similar improvement might be handy. Mind creating a JIRA ticket for this change, please?

[GitHub] [logging-log4j2] Francis-FY opened a new pull request #676: LOG4J2-3304 - Fix issue that the initialize status of LogManager is not set when factoryClassName is present and instantiate succe

2021-12-30 Thread GitBox
Francis-FY opened a new pull request #676: URL: https://github.com/apache/logging-log4j2/pull/676 Fix the issue that the initialize status of LogManager is not set when logger factoryClassName is present and instantiate successfully, see [LOG4J2-3304](https://issues.apache.org/jira/browse/

[GitHub] [logging-log4j2] garydgregory commented on pull request #676: LOG4J2-3304 - Fix issue that the initialize status of LogManager is not set when factoryClassName is present and instantiate suc

2021-12-30 Thread GitBox
garydgregory commented on pull request #676: URL: https://github.com/apache/logging-log4j2/pull/676#issuecomment-1003084416 Hi @Francis-FY Thank you for your PR? How could this be validated in a unit test? -- This is an automated message from the Apache Git Service. To respond to t

[GitHub] [logging-log4j2] garydgregory edited a comment on pull request #676: LOG4J2-3304 - Fix issue that the initialize status of LogManager is not set when factoryClassName is present and instanti

2021-12-30 Thread GitBox
garydgregory edited a comment on pull request #676: URL: https://github.com/apache/logging-log4j2/pull/676#issuecomment-1003084416 Hi @Francis-FY Thank you for your PR. How could this be validated in a unit test? -- This is an automated message from the Apache Git Service. To respo

[GitHub] [logging-log4j2] rgoers commented on pull request #676: LOG4J2-3304 - Fix issue that the initialize status of LogManager is not set when factoryClassName is present and instantiate successfu

2021-12-30 Thread GitBox
rgoers commented on pull request #676: URL: https://github.com/apache/logging-log4j2/pull/676#issuecomment-1003093157 I'm having a hard time understanding how this relates to Log4j's Spring Boot support. -- This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [logging-log4j2] Francis-FY commented on pull request #676: LOG4J2-3304 - Fix issue that the initialize status of LogManager is not set when factoryClassName is present and instantiate succe

2021-12-30 Thread GitBox
Francis-FY commented on pull request #676: URL: https://github.com/apache/logging-log4j2/pull/676#issuecomment-1003100483 @rgoers The method getEvnironment of SpringEnvironmentHolder will check if LogManager is initialized through LogManagerStatus.isInitialized(). You can run a unit test

[GitHub] [logging-log4j2] ramananravi commented on pull request #671: [LOG4J2-3303] Add support for FileSize in TB

2021-12-30 Thread GitBox
ramananravi commented on pull request #671: URL: https://github.com/apache/logging-log4j2/pull/671#issuecomment-1003111212 Edited the PR description to add reference to jira ticket (LOG4J2-3303). Could see some checks failing now (previously all checks passed). Are these intermittent failu

[GitHub] [logging-log4j2] ramananravi edited a comment on pull request #671: [LOG4J2-3303] Add support for FileSize in TB

2021-12-30 Thread GitBox
ramananravi edited a comment on pull request #671: URL: https://github.com/apache/logging-log4j2/pull/671#issuecomment-1003111212 Edited the PR title to add reference to jira ticket (LOG4J2-3303). Could see some checks failing now (previously all checks passed). Are these intermittent fail

[GitHub] [logging-chainsaw] rm5248 merged pull request #10: Copied the relevant log4j1 extras into chainsaw

2021-12-31 Thread GitBox
rm5248 merged pull request #10: URL: https://github.com/apache/logging-chainsaw/pull/10 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notificati

[GitHub] [logging-log4j2] rgoers opened a new pull request #677: Log4 j2 3301

2021-12-31 Thread GitBox
rgoers opened a new pull request #677: URL: https://github.com/apache/logging-log4j2/pull/677 Reverts to using prior logic but prevents endless conversion. Note that the StrSubstitutor implementation had some issues - it still could get a StackOverflowError, just in a different scenario. S

[GitHub] [logging-log4cxx] swebb2066 commented on pull request #90: [LOGCXX-546] Prevent unnecessary locking when determining the logging level

2021-12-31 Thread GitBox
swebb2066 commented on pull request #90: URL: https://github.com/apache/logging-log4cxx/pull/90#issuecomment-1003503632 Is this good to merge? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [logging-log4cxx] swebb2066 commented on pull request #90: [LOGCXX-546] Prevent unnecessary locking when determining the logging level

2021-12-31 Thread GitBox
swebb2066 commented on pull request #90: URL: https://github.com/apache/logging-log4cxx/pull/90#issuecomment-1003504095 Merged using https://gitbox.apache.org -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [logging-log4cxx] swebb2066 closed pull request #90: [LOGCXX-546] Prevent unnecessary locking when determining the logging level

2021-12-31 Thread GitBox
swebb2066 closed pull request #90: URL: https://github.com/apache/logging-log4cxx/pull/90 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifica

[GitHub] [logging-log4cxx] swebb2066 commented on pull request #93: Add a unit test for the automatic configuration feature

2021-12-31 Thread GitBox
swebb2066 commented on pull request #93: URL: https://github.com/apache/logging-log4cxx/pull/93#issuecomment-1003508384 Merged as 7fedcdbf using https://gitbox.apache.org -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [logging-log4cxx] swebb2066 closed pull request #93: Add a unit test for the automatic configuration feature

2021-12-31 Thread GitBox
swebb2066 closed pull request #93: URL: https://github.com/apache/logging-log4cxx/pull/93 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifica

[GitHub] [logging-log4cxx] swebb2066 opened a new pull request #94: LOGCXX-546] Prevent serialization of a multi-threaded application

2021-12-31 Thread GitBox
swebb2066 opened a new pull request #94: URL: https://github.com/apache/logging-log4cxx/pull/94 when using disabled logging statements. throughputtests output after applyinh these changes: ```Benchmarking library only(no writing out):

[GitHub] [logging-log4cxx] rm5248 commented on a change in pull request #93: Add a unit test for the automatic configuration feature

2022-01-01 Thread GitBox
rm5248 commented on a change in pull request #93: URL: https://github.com/apache/logging-log4cxx/pull/93#discussion_r776895762 ## File path: src/main/cpp/aprinitializer.cpp ## @@ -127,3 +127,21 @@ void APRInitializer::unregisterCleanup(FileWatchdog* watchdog) } } +v

[GitHub] [logging-log4cxx] rm5248 commented on a change in pull request #86: Update log4cxx.h.in

2022-01-01 Thread GitBox
rm5248 commented on a change in pull request #86: URL: https://github.com/apache/logging-log4cxx/pull/86#discussion_r773494809 ## File path: src/main/include/log4cxx/log4cxx.h.in ## @@ -29,10 +29,14 @@ #define LOG4CXX_VERSION_MINOR @log4cxx_VERSION_MINOR@ #define LOG4CXX_VERS

[GitHub] [logging-log4cxx] rm5248 commented on pull request #93: Add a unit test for the automatic configuration feature

2022-01-01 Thread GitBox
rm5248 commented on pull request #93: URL: https://github.com/apache/logging-log4cxx/pull/93#issuecomment-1003560920 Apparently I forgot to submit my review comments, but those are above. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [logging-log4cxx] rm5248 commented on a change in pull request #90: [LOGCXX-546] Prevent unnecessary locking when determining the logging level

2022-01-01 Thread GitBox
rm5248 commented on a change in pull request #90: URL: https://github.com/apache/logging-log4cxx/pull/90#discussion_r776893142 ## File path: src/main/include/log4cxx/logger.h ## @@ -617,13 +617,13 @@ class LOG4CXX_EXPORT Logger : @throws RuntimeException if al

<    38   39   40   41   42   43   44   45   46   47   >