robertwb commented on a change in pull request #16499:
URL: https://github.com/apache/beam/pull/16499#discussion_r793159861
##########
File path:
learning/katas/java/util/src/org/apache/beam/learning/katas/util/Log.java
##########
@@ -70,6 +70,7 @@ public void processElement(
}
LOG.info(message);
+ System.out.println(message)
Review comment:
Generally logging only shows up on stderr. (Does the playground not show
stderr?)
Given this transform is explicitly about logging, it feels really odd to
also print to stdout. We could decide to use a stdout-printing transform rather
than a logging transform for the katas.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
