(samza) 01/01: Address comments

[bharathkk]

This is an automated email from the ASF dual-hosted git repository.

bharathkk pushed a commit to branch SAMZA-2763
in repository https://gitbox.apache.org/repos/asf/samza.git

commit 740fd99d95fb2ca37f0fcb338b5d224f05ced7fe
Author: Bharath Kumarasubramanian <bharat...@apache.org>
AuthorDate: Mon Nov 20 14:01:01 2023 -0800

    Address comments
---
 .../src/main/java/org/apache/samza/config/ShellCommandConfig.java   | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/samza-core/src/main/java/org/apache/samza/config/ShellCommandConfig.java 
b/samza-core/src/main/java/org/apache/samza/config/ShellCommandConfig.java
index 3cbbf0321..90780c81d 100644
--- a/samza-core/src/main/java/org/apache/samza/config/ShellCommandConfig.java
+++ b/samza-core/src/main/java/org/apache/samza/config/ShellCommandConfig.java
@@ -106,6 +106,9 @@ public class ShellCommandConfig extends MapConfig {
     return Optional.ofNullable(getFinalJvmOptions(taskOpts, 
autosizingContainerMaxHeap));
   }
 
+  /**
+   * Returns the worker opts for the application if available.
+   */
   public Optional<String> getWorkerOpts() {
     String autosizingWorkerHeapMb = 
get(JobConfig.JOB_AUTOSIZING_WORKER_MAX_HEAP_MB);
     String workerOpts = get(ShellCommandConfig.WORKER_JVM_OPTS);
@@ -120,6 +123,9 @@ public class ShellCommandConfig extends MapConfig {
     return 
Optional.ofNullable(get(ShellCommandConfig.ADDITIONAL_CLASSPATH_DIR));
   }
 
+  /**
+   * Returns the final JVM options by applying the heap override if available 
to the jvm opts
+   */
   @VisibleForTesting
   String getFinalJvmOptions(String jvmOpts, String maxHeapOverride) {
     String finalJvmOpts = jvmOpts;