Re: [PR] Feature: Implement pagination in new `PaginatedHttpOperator` + refactor `SimpleHttpOperator` [airflow]

2023-10-24 Thread via GitHub
jens-scheffler-bosch commented on code in PR #34669: URL: https://github.com/apache/airflow/pull/34669#discussion_r1370673376 ## docs/apache-airflow-providers-http/operators.rst: ## @@ -118,3 +118,24 @@ Here we pass form data to a ``POST`` operation which is equal to a usual fo

Re: [PR] Feature: Implement pagination in new `PaginatedHttpOperator` + refactor `SimpleHttpOperator` [airflow]

2023-10-12 Thread via GitHub
Joffreybvn commented on PR #34669: URL: https://github.com/apache/airflow/pull/34669#issuecomment-1759136750 Everything in one single Http Operator is indeed simpler ! From a user point of view, I get the docstring on my IDE and can opt-in to a pagination feature just by providing the requi

Re: [PR] Feature: Implement pagination in new `PaginatedHttpOperator` + refactor `SimpleHttpOperator` [airflow]

2023-10-12 Thread via GitHub
Joffreybvn commented on PR #34669: URL: https://github.com/apache/airflow/pull/34669#issuecomment-1759086793 @hussein-awala Could you review this one ? (as you reviewed #34606 ) By 'making generic', you mean putting a generic name on it ? -- This is an automated message from the Apache