chore: Fix review comments

Signed-off-by: Javier Aliaga <[email protected]>

Author: javier-aliaga

examples/src/main/java/io/dapr/examples/workflows/crossapp/App2TransformActivity.java

@@ -21,13 +21,14 @@
  * This activity is called cross-app from the main workflow.
  */
 public class App2TransformActivity implements WorkflowActivity {
-    @Override
-    public Object run(WorkflowActivityContext context) {
-        String input = context.getInput(String.class);
-        System.out.println("=== App2: TransformActivity called ===");
-        System.out.println("Input: " + input);
-        String result = input.toUpperCase() + " [TRANSFORMED BY APP2]";
-        System.out.println("Output: " + result);
-        return result;
-    }
+  @Override
+  public Object run(WorkflowActivityContext context) {
+    String input = context.getInput(String.class);
+    var logger = context.getLogger();
+    logger.info("=== App2: TransformActivity called ===");
+    logger.info("Input: {}", input);
+    String result = input.toUpperCase() + " [TRANSFORMED BY APP2]";
+    logger.info("Output: {}", result);
+    return result;
+  }
 }

examples/src/main/java/io/dapr/examples/workflows/crossapp/App3FinalizeActivity.java

@@ -24,10 +24,11 @@ public class App3FinalizeActivity implements WorkflowActivity {
     @Override
     public Object run(WorkflowActivityContext context) {
         String input = context.getInput(String.class);
-        System.out.println("=== App3: FinalizeActivity called ===");
-        System.out.println("Input: " + input);
+        var logger = context.getLogger();
+        logger.info("=== App3: FinalizeActivity called ===");
+        logger.info("Input: {}", input);
         String result = input + " [FINALIZED BY APP3]";
-        System.out.println("Output: " + result);
+        logger.info("Output: {}", result);
         return result;
     }
 }

examples/src/main/java/io/dapr/examples/workflows/crossapp/CrossAppWorkflow.java

@@ -25,18 +25,19 @@ public class CrossAppWorkflow implements Workflow {
   @Override
   public WorkflowStub create() {
     return ctx -> {
-      ctx.getLogger().info("=== WORKFLOW STARTING ===");
-      ctx.getLogger().info("Starting CrossAppWorkflow: " + ctx.getName());
-      ctx.getLogger().info("Workflow name: " + ctx.getName());
-      ctx.getLogger().info("Workflow instance ID: " + ctx.getInstanceId());
+      var logger = ctx.getLogger();
+      logger.info("=== WORKFLOW STARTING ===");
+      logger.info("Starting CrossAppWorkflow: {}", ctx.getName());
+      logger.info("Workflow name: {}", ctx.getName());
+      logger.info("Workflow instance ID: {}", ctx.getInstanceId());
 
       String input = ctx.getInput(String.class);
-      ctx.getLogger().info("CrossAppWorkflow received input: " + input);
-      ctx.getLogger().info("Workflow input: " + input);
+      logger.info("CrossAppWorkflow received input: {}", input);
+      logger.info("Workflow input: {}", input);
 
       // Call an activity in another app by passing in an active appID to the WorkflowTaskOptions
-      ctx.getLogger().info("Calling cross-app activity in 'app2'...");
-      ctx.getLogger().info("About to call cross-app activity in app2...");
+      logger.info("Calling cross-app activity in 'app2'...");
+      logger.info("About to call cross-app activity in app2...");
       String crossAppResult = ctx.callActivity(
           App2TransformActivity.class.getName(),
           input,
@@ -45,19 +46,18 @@ public WorkflowStub create() {
       ).await();
 
       // Call another activity in a different app
-      ctx.getLogger().info("Calling cross-app activity in 'app3'...");
-      ctx.getLogger().info("About to call cross-app activity in app3...");
+      logger.info("Calling cross-app activity in 'app3'...");
+      logger.info("About to call cross-app activity in app3...");
       String finalResult = ctx.callActivity(
           App3FinalizeActivity.class.getName(),
           crossAppResult,
           new WorkflowTaskOptions("app3"),
           String.class
       ).await();
-      ctx.getLogger().info("Final cross-app activity result: " + finalResult);
-      ctx.getLogger().info("Final cross-app activity result: " + finalResult);
-      
-      ctx.getLogger().info("CrossAppWorkflow finished with: " + finalResult);
-      ctx.getLogger().info("=== WORKFLOW COMPLETING WITH: " + finalResult + " ===");
+      logger.info("Final cross-app activity result: {}", finalResult);
+
+      logger.info("CrossAppWorkflow finished with: {}", finalResult);
+      logger.info("=== WORKFLOW COMPLETING WITH: {} ===", finalResult);
       ctx.complete(finalResult);
     };
   }

sdk-autogen/pom.xml

@@ -22,7 +22,7 @@
     <protobuf.input.directory>${project.build.directory}/proto</protobuf.input.directory>
     <maven.deploy.skip>false</maven.deploy.skip>
     <grpc.version>1.69.0</grpc.version>
-    <protocCommand>protoc</protocCommand>
+    <protocCommand>java-sdk-protoc</protocCommand>
     <protobuf.version>3.25.5</protobuf.version>
   </properties>
 
@@ -132,7 +132,7 @@
               <goal>run</goal>
             </goals>
             <configuration>
-              <protocCommand>protoc</protocCommand>
+              <protocCommand>java-sdk-protoc</protocCommand>
               <protocVersion>${protobuf.version}</protocVersion>
               <protocArtifact>com.google.protobuf:protoc:3.25.5</protocArtifact>
               <addProtoSources>inputs</addProtoSources>