8000 use NonNegativeInt for backfill_id by rawwar · Pull Request #49691 · apache/airflow · GitHub
[go: up one dir, main page]
Skip to content

use NonNegativeInt for backfill_id #49691

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Apr 24, 2025
Merged

use NonNegativeInt for backfill_id #49691

merged 3 commits into from
Apr 24, 2025

Conversation

rawwar
Copy link
Contributor
@rawwar rawwar commented Apr 24, 2025

Noticed test failures in another PR - https://github.com/apache/airflow/actions/runs/14634141646/job/41061963159?pr=49599

backfill_id is defined as an integer in DB. So, making it NonNegativeInt

@boring-cyborg boring-cyborg bot added area:airflow-ctl area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels Apr 24, 2025
@rawwar rawwar requested a review from pierrejeambrun April 24, 2025 07:57
@rawwar rawwar marked this pull request as ready for review April 24, 2025 07:58
@rawwar rawwar marked this pull request as draft April 24, 2025 08:45
@rawwar rawwar force-pushed the kalyan/AIP-84/backfill/fix_backfill_id branch from 4e92899 to f205a7e Compare April 24, 2025 11:05
@rawwar rawwar marked this pull request as ready for review April 24, 2025 11:49
@pierrejeambrun pierrejeambrun added the type:bug-fix Changelog: Bug Fixes label Apr 24, 2025
@pierrejeambrun pierrejeambrun added the backport-to-v3-0-test Mark PR with this label to backport to v3-0-test branch label Apr 24, 2025
@rawwar rawwar merged commit e564f1b into apache:main Apr 24, 2025
110 of 173 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 24, 2025
@rawwar
[v3-0-test] use NonNegativeInt for backfill_id (#49691)
3ba794d 
* use NonNegativeInt for backfill_id

* fix aws failing test

* update BackfillResponse
(cherry picked from commit e564f1b)

Co-authored-by: Kalyan R <kalyan.ben10@live.com>
@github-actions GitHub Actions
Copy link

Backport successfully created: v3-0-test

Status Branch Result
✅ v3-0-test PR Link

@rawwar rawwar deleted the kalyan/AIP-84/backfill/fix_backfill_id branch April 24, 2025 13:13
pierrejeambrun pushed a commit that referenced this pull request Apr 24, 2025
@github-actions @rawwar
[v3-0-test] use NonNegativeInt for backfill_id (#49691) (#49716)
995fcde 
* use NonNegativeInt for backfill_id

* fix aws failing test

* update BackfillResponse
(cherry picked from commit e564f1b)

Co-authored-by: Kalyan R <kalyan.ben10@live.com>
kaxil pushed a commit that referenced this pull request Apr 24, 2025
@github-actions @rawwar @kaxil
Use NonNegativeInt for backfill_id (#49691) (#49716)
f03f1e9 
* use NonNegativeInt for backfill_id

* fix aws failing test

* update BackfillResponse
(cherry picked from commit e564f1b)

Co-authored-by: Kalyan R <kalyan.ben10@live.com>
prabhusneha pushed a commit to astronomer/airflow that referenced this pull request Apr 25, 2025
@rawwar
use NonNegativeInt for backfill_id (apache#49691)
fe20d21 
* use NonNegativeInt for backfill_id

* fix aws failing test

* update BackfillResponse
@kaxil kaxil mentioned this pull request Jun 10, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pierrejeambrun pierrejeambrun pierrejeambrun approved these changes

@bbovenzi bbovenzi Awaiting requested review from bbovenzi bbovenzi is a code owner

@ryanahamilton ryanahamilton Awaiting requested review from ryanahamilton ryanahamilton is a code owner

@jscheffl jscheffl Awaiting requested review from jscheffl jscheffl is a code owner

@shubhamraj-git shubhamraj-git Awaiting requested review from shubhamraj-git shubhamraj-git is a code owner

@vincbeck vincbeck Awaiting requested review from vincbeck vincbeck is a code owner

@bugraoz93 bugraoz93 Awaiting requested review from bugraoz93 bugraoz93 is a code owner

@kaxil kaxil Awaiting requested review from kaxil kaxil is a code owner

@potiuk potiuk Awaiting requested review from potiuk potiuk is a code owner

@ephraimbuddy ephraimbuddy Awaiting requested review from ephraimbuddy ephraimbuddy is a code owner

@jason810496 jason810496 Awaiting requested review from jason810496 jason810496 is a code owner

@XD-DENG XD-DENG Awaiting requested review from XD-DENG XD-DENG is a code owner

@ashb ashb Awaiting requested review from ashb ashb is a code owner

@eladkal eladkal Awaiting requested review from eladkal eladkal is a code owner

@o-nikolas o-nikolas Awaiting requested review from o-nikolas o-nikolas is a code owner

Assignees
No one assigned
Labels
area:airflow-ctl area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. backport-to-v3-0-test Mark PR with this label to backport to v3-0-test branch type:bug-fix Changelog: Bug Fixes
Projects
None yet
Milestone
Airflow 3.0.1
Development

Successfully merging this pull request may close these issues.
2 participants
@rawwar @pierrejeambrun
0