Joffreybvn commented on PR #34669: URL: https://github.com/apache/airflow/pull/34669#issuecomment-1739992860
Thanks for early review ! Mypy is indeed still complaining., I'll take time tomorrow to fix it. **About your last point**: Yes, i considered the `response_filter`. Initially, I wrote a simple procedure embedded into a PythonOperator; which was using the hook directly. I'd not put that into the `response_filter` function because: - It's simpler to just use a PythonOperator and the hook (instead of bothering with the Operator for the first call) - The response filter is very convenient for filtering responses. _It would be great to receive a list of Responses and filter out the cursors from the real data_. especially if I can use it to receive a list of Response made by a complex HttpOperator object. In my opinion, it is too hacky to try to inject big things into it, especially -- 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]
