kolfild26 commented on PR #27319:
URL: https://github.com/apache/airflow/pull/27319#issuecomment-1302713101

   @bdsoha 
   
   > You can accomplish the same thing using regex:
   > 
   > `import re`
   > `code = re.search("^ORA-(\\d+):.+", "ORA-28365: wallet is not 
open").group(1)`
   
   Yes, but above were the suggestion by @uranusjr  not to split so much at 
all, no matter which way is used:
   
   > Why not just push str(e) and move extraction logic into downstream tasks, 
for example
   
   I prefer the way @bdsoha (to split). So if no objections to this I would 
commit `re.search`
   


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