Nike-Inc / brickflow

Pythonic Programming Framework to orchestrate jobs in Databricks Workflow

https://engineering.nike.com/brickflow/

Apache License 2.0

187 stars 44 forks source link

[fix] Adjust logic for applying outcome for IF_ELSE_CONDITION tasks #173

Closed mikita-sakalouski closed 1 week ago

mikita-sakalouski commented 1 week ago

Description

Add check that outcome is applied on correct task from dependencies
Get correct outcome based on task key and not just first one

Related Issue

172 #171

Motivation and Context

Get correct behavior

How Has This Been Tested?

Added more tasks into sample workflow for testing

Screenshots (if appropriate):

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

mikita-sakalouski commented 1 week ago

Depends on #168 . Please firstly review and merge PR #168