zhuzhurk commented on code in PR #24747:
URL: https://github.com/apache/flink/pull/24747#discussion_r1590940937


##########
flink-clients/src/main/java/org/apache/flink/client/deployment/application/executors/EmbeddedExecutor.java:
##########
@@ -163,7 +163,7 @@ private static CompletableFuture<JobID> submitJob(
             final Time rpcTimeout) {
         checkNotNull(jobGraph);
 
-        LOG.info("Submitting Job with JobId={}.", jobGraph.getJobID());
+        LOG.info("Submitting job '{}' ({}).", jobGraph.getName(), 
jobGraph.getJobID());

Review Comment:
   The job submission log of application mode is already a bit redundant. 
Therefore I prefer to not further complicate it. 
   
   e.g.
   ```
   2024-05-06 20:19:20,853 [flink-akka.actor.default-dispatcher-16] INFO  
org.apache.flink.client.deployment.application.executors.EmbeddedExecutor [] - 
Job 2214f37f6f124fadb641e22182325897 is submitted.
   2024-05-06 20:19:20,854 [flink-akka.actor.default-dispatcher-16] INFO  
org.apache.flink.client.deployment.application.executors.EmbeddedExecutor [] - 
Submitting Job with JobId=2214f37f6f124fadb641e22182325897.
   2024-05-06 20:19:21,141 [flink-akka.actor.default-dispatcher-5] INFO  
org.apache.flink.runtime.dispatcher.StandaloneDispatcher     [] - Received 
JobGraph submission 'Flink Streaming Job' (2214f37f6f124fadb641e22182325897).
   2024-05-06 20:19:21,177 [flink-akka.actor.default-dispatcher-5] INFO  
org.apache.flink.runtime.dispatcher.StandaloneDispatcher     [] - Submitting 
job 'Flink Streaming Job' (2214f37f6f124fadb641e22182325897).
   ```
   



-- 
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: issues-unsubscr...@flink.apache.org

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

Reply via email to