herbherbherb opened a new pull request, #28079:
URL: https://github.com/apache/flink/pull/28079

   ## What is the purpose of the change                                         
                                                                                
                                                         
                                         
     When `AdaptiveScheduler`'s `CreatingExecutionGraph` state receives a       
                                                                                
                                                           
     non-null throwable from the `ExecutionGraphWithVertexParallelism`          
                                                                                
                                                           
     future, Flink fails to construct/restore the execution graph and           
                                                                                
                                                           
     immediately transitions the job to FAILED. Today the root-cause            
                                                                                
                                                           
     throwable is logged at INFO, which makes incidents harder to detect and
     triage in environments that rely on log severity for alerting.             
                                                                                
                                                           
                                                                                
                                                                                
                                                           
     This PR promotes that single failure log call from INFO to ERROR.          
                                                                                
                                                           
                                                                                
                                                                                
                                                           
     ## Brief change log                                             
     - In `CreatingExecutionGraph#handleExecutionGraphCreation`, change the
       log call in the `throwable != null` branch from `info(...)` to           
                                                                                
                                                         
       `error(...)`. No other call sites are changed.                
                                                             


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