Abacn commented on PR #27879:
URL: https://github.com/apache/beam/pull/27879#issuecomment-1668425917

   The line that will cause problem is here:
   
   
https://github.com/bvolpato/beam/blob/9e3d8d3c5719bbe8052b10c19be028d6b93f3eaf/runners/google-cloud-dataflow-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/util/MemoryMonitor.java#L245
   
   this change alone won't be effective to eliminate the issue when using an 
outdated workerJar
   
   to me this is a non-issue / documentation issue. We need to bump 
https://github.com/apache/beam/blob/3b112e8bd8da50d934955d64f2fd1e1396d3fe01/runners/google-cloud-dataflow-java/build.gradle#L56
 here


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

Reply via email to