[GitHub] [beam] rohdesamuel commented on a change in pull request #13210: [BEAM-10994] Update hot key detection log message

[GitBox <gi...@apache.org>]

rohdesamuel commented on a change in pull request #13210:
URL: https://github.com/apache/beam/pull/13210#discussion_r513741622



##########
File path: runners/google-cloud-dataflow-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/HotKeyLogger.java
##########
@@ -52,7 +52,9 @@ public void logHotKeyDetection(String userStepName, Duration hotKeyAge) {
     LOG.warn(
         "A hot key was detected in step '{}' with age of '{}'. This is "
             + "a symptom of key distribution being skewed. To fix, please inspect your data and "
-            + "pipeline to ensure that elements are evenly distributed across your key space.",
+            + "pipeline to ensure that elements are evenly distributed across your key space. If "
+            + "you want to log the plain-text key to Cloud Logging please re-run with the "
+            + "`hotKeyLoggingEnabled` pipeline option.",

Review comment:
       Sure, I added a link to the Cloud docs about specifying pipeline options. This generalizes across different languages and is a good how-to guide.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org