PnPie opened a new pull request #562: [GRIFFIN-316] Fix job exception handling
URL: https://github.com/apache/griffin/pull/562
 
 
   **What changes were proposed in this pull request?**
   
   Currently we are using Try instance to represent the results of a DQ job, 
whether succeeded or failed. But as we are only wrapping the Boolean result by 
applying "Try" at the most outside level, the underlying failure would not be 
able to caught and it would always return "Success" even if exception got.
   
   This is to modify all the underlying execute/doExecute methods of a DQ job, 
by handling exception with "Try" instances so that it could be passed properly 
to users when things get wrong.
   
   **Does this PR introduce any user-facing change?**
   No.
   
   **How was this patch tested?**
   Griffin test suite.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to