Skip to content

Simplify default rows limit return result - #64183

Merged
gopidesupavan merged 2 commits into
apache:mainfrom
gopidesupavan:fix-analyticsoperator-limit
Mar 24, 2026
Merged

Simplify default rows limit return result#64183
gopidesupavan merged 2 commits into
apache:mainfrom
gopidesupavan:fix-analyticsoperator-limit

Conversation

@gopidesupavan

Copy link
Copy Markdown
Member

Simplify the limit logic on enginee level itself to safely return the results instead of an operator level.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@gopidesupavan
gopidesupavan force-pushed the fix-analyticsoperator-limit branch from 15aeedb to 6227603 Compare March 24, 2026 19:19
Comment thread providers/common/sql/src/airflow/providers/common/sql/operators/analytics.py Outdated
@gopidesupavan
gopidesupavan merged commit 8d43942 into apache:main Mar 24, 2026
94 checks passed
@gopidesupavan
gopidesupavan deleted the fix-analyticsoperator-limit branch March 24, 2026 20:31
@jscheffl

Copy link
Copy Markdown
Contributor

Args, damn, this was merged while I was waiting on CI to merge changelogs of providers release... damn now need to start over...

@gopidesupavan

Copy link
Copy Markdown
Member Author

Args, damn, this was merged while I was waiting on CI to merge changelogs of providers release... damn now need to start over...

oops sorry ...

vatsrahul1001 pushed a commit that referenced this pull request Mar 25, 2026
* Simplify default rows limit return result

* Resolve comments
aaron-y-chen pushed a commit to aaron-y-chen/airflow that referenced this pull request Mar 30, 2026
* Simplify default rows limit return result

* Resolve comments
Suraj-kumar00 pushed a commit to Suraj-kumar00/airflow that referenced this pull request Apr 7, 2026
* Simplify default rows limit return result

* Resolve comments
abhijeets25012-tech pushed a commit to abhijeets25012-tech/airflow that referenced this pull request Apr 9, 2026
* Simplify default rows limit return result

* Resolve comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants