Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-12-03 Thread via GitHub
timoninmaxim merged PR #11615: URL: https://github.com/apache/ignite/pull/11615 -- 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-unsubscr..

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-25 Thread via GitHub
timoninmaxim commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1856850659 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -17,39 +17,130 @@ package org.apache.ignite.logger.java; +import java.

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-13 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1840892701 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -17,39 +17,110 @@ package org.apache.ignite.logger.java; +import java.io

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-13 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r184015 ## modules/core/src/test/java/org/apache/ignite/testframework/junits/logger/GridTestLog4jLoggerSelfTest.java: ## @@ -64,28 +71,89 @@ public static void afterTests()

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-13 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1840888342 ## modules/core/src/test/java/org/apache/ignite/testframework/junits/logger/GridTestLog4jLoggerSelfTest.java: ## @@ -64,28 +71,89 @@ public static void afterTests()

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-13 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1840888342 ## modules/core/src/test/java/org/apache/ignite/testframework/junits/logger/GridTestLog4jLoggerSelfTest.java: ## @@ -64,28 +71,89 @@ public static void afterTests()

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-13 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1840888342 ## modules/core/src/test/java/org/apache/ignite/testframework/junits/logger/GridTestLog4jLoggerSelfTest.java: ## @@ -64,28 +71,89 @@ public static void afterTests()

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-13 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r184015 ## modules/core/src/test/java/org/apache/ignite/testframework/junits/logger/GridTestLog4jLoggerSelfTest.java: ## @@ -64,28 +71,89 @@ public static void afterTests()

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-11 Thread via GitHub
timoninmaxim commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1836102934 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -17,39 +17,110 @@ package org.apache.ignite.logger.java; +import java.

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-08 Thread via GitHub
timoninmaxim commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1834421285 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -84,4 +153,5 @@ public void testLogInitialize() throws Exception {

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-08 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1834409657 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -17,31 +17,154 @@ package org.apache.ignite.logger.java; +import java.io

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-08 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1834409657 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -17,31 +17,154 @@ package org.apache.ignite.logger.java; +import java.io

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-08 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1834361455 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -84,4 +207,13 @@ public void testLogInitialize() throws Exception {

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-08 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1834361455 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -84,4 +207,13 @@ public void testLogInitialize() throws Exception {

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-08 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1834361455 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -84,4 +207,13 @@ public void testLogInitialize() throws Exception {

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-08 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1834361455 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -84,4 +207,13 @@ public void testLogInitialize() throws Exception {

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-08 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1834361455 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -84,4 +207,13 @@ public void testLogInitialize() throws Exception {

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-06 Thread via GitHub
timoninmaxim commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1832180105 ## modules/core/src/test/java/org/apache/ignite/logger/java/JavaLoggerTest.java: ## @@ -17,31 +17,154 @@ package org.apache.ignite.logger.java; +import java.

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-05 Thread via GitHub
chesnokoff closed pull request #11615: IGNITE-23472 Fix JavaLogger URL: https://github.com/apache/ignite/pull/11615 -- 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 unsubscri

Re: [PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-11-01 Thread via GitHub
chesnokoff commented on code in PR #11615: URL: https://github.com/apache/ignite/pull/11615#discussion_r1825648205 ## modules/core/src/main/java/org/apache/ignite/logger/java/JavaLogger.java: ## @@ -134,7 +138,7 @@ public class JavaLogger implements IgniteLoggerEx { * Crea

[PR] IGNITE-23472 Fix JavaLogger [ignite]

2024-10-23 Thread via GitHub
chesnokoff opened a new pull request, #11615: URL: https://github.com/apache/ignite/pull/11615 JavaLogger.configure method is invoked only once for all instances and it configures cfg field which was not static. Because of that only one instance had correct value and other instances had cfg