Skip to content
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.

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

AIP-83 question 4.b. manual run data interval behavior #46197

Closed
dstandish opened this issue Jan 28, 2025 · 2 comments · Fixed by #46512
Closed

AIP-83 question 4.b. manual run data interval behavior #46197

dstandish opened this issue Jan 28, 2025 · 2 comments · Fixed by #46512
Assignees
Labels
kind:meta High-level information important to the community

Comments

@dstandish
Copy link
Contributor

dstandish commented Jan 28, 2025

Manual runs, schedule-driven dag

If logical date null, there should be no data interval; if logical date not null, data interval should be as normal.

If logical date is null, accessing data interval in context or templates should result in a key error.

Manual runs, dag is asset driven

There will be no data interval.

Accessing data interval in context or templates should result in a key error.

@dstandish dstandish added the kind:meta High-level information important to the community label Jan 28, 2025
@dstandish dstandish moved this to Todo in AIP-83 amendment Jan 28, 2025
@sunank200 sunank200 self-assigned this Jan 31, 2025
@phanikumv phanikumv moved this from Todo to In Progress in AIP-83 amendment Feb 3, 2025
@sunank200
Copy link
Collaborator

sunank200 commented Feb 5, 2025

I am pushing changes to null-logical-date-for-ui-trigger branch.

@sunank200
Copy link
Collaborator

PR: 46512

@github-project-automation github-project-automation bot moved this from In Progress to Done in AIP-83 amendment Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:meta High-level information important to the community
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants