kaxil commented on PR #71947: URL: https://github.com/apache/airflow/pull/71947#issuecomment-6037402584
> I am not sure i understand the question 'Who needs these four operators?' :) These operators cover existing API and can be used instead of just calling it directly in system test Why exactly it was marked as draft? Mainly asking because the Operators in these PRs are literally just one operator per hook method. So wanted to ensure there is a real need of this and we don't just create 1 operator per 1 hook method without a reason. I think I detailed it on why it was converted to draft in previous comment :) > The PR also changes behavior the description doesn't mention: list_reports now returns list[dict] instead of the raw response, the sensor raises on FAILED where it returned True, and DownloadReportOperator now raises unless the status is DONE. Is there an issue or a user asking for the operators, and a reported problem behind the behavior changes? Converting to draft until there 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]
