Eason09053360 commented on PR #73132: URL: https://github.com/apache/airflow/pull/73132#issuecomment-6206385303
> Should we wrap the `bulk()` call in try/except so this surfaces as a `SystemExit` message instead of a traceback? It wouldn't change the rejection itself the whole-batch behaviour is a server-side contract question, but the command reports every other failure that way. Thanks for reviewing! Test that traceback comes from this branch predating #73307, which made `safe_call_command` catch non-JSON errors such as the bare 500 a nonexistent `team_name` gets. Rebased onto main, the same import exits 1 with `Server response error: …`, so I've left `_import_helper` as is. -- 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]
