Merge pull request #977: [beam] do not use String.format for logging errors in ProximaIO

je-ik · web-flow · commit e2de2ff705a8 · 2025-04-14T09:27:25.000+02:00
diff --git a/beam/core/src/main/java/cz/o2/proxima/beam/io/ProximaIO.java b/beam/core/src/main/java/cz/o2/proxima/beam/io/ProximaIO.java
@@ -160,7 +160,7 @@ public void processElement(@Element StreamElement element) {
                 (succ, error) -> {
                   writeResult.complete(Pair.of(succ, error));
                   if (error != null) {
-                    log.error(String.format("Unable to write element [%s].", element), error);
+                    log.error("Unable to write element [{}]", element, error);
                   }
                 });
           });

Original file line number	Diff line number	Diff line change
`@@ -160,7 +160,7 @@ public void processElement(@Element StreamElement element) {`
`160`	`160`	`(succ, error) -> {`
`161`	`161`	`writeResult.complete(Pair.of(succ, error));`
`162`	`162`	`if (error != null) {`
`163`		`- log.error(String.format("Unable to write element [%s].", element), error);`
	`163`	`+ log.error("Unable to write element [{}]", element, error);`
`164`	`164`	`}`
`165`	`165`	`});`
`166`	`166`	`});`