The GitHub Actions job "Tests (AMD)" on airflow.git/backport-b767477-v3-3-test 
has failed.
Run started by GitHub user bbovenzi (triggered by bbovenzi).

Head commit for run:
7cca868afbfa7f2053e2a249406ec57919780d27 / Wei Lee <[email protected]>
[v3-3-test] Fix partition key display and input handling (#69702)

* UI: Fix partition key display and input handling

Non-partitioned Dag runs showed a dangling "Partition key:" label, an
empty partition key typed then cleared was sent to the trigger/materialize
APIs and rejected with 400, the trigger form exposed the field even for
non-partitioned Dags, the manual asset-event partition key used a
JsonEditor for a plain string, and the runs table hid the partition_key
column even when its filter was shown for a partitioned Dag. Also unify
the partition key label so cron/runtime-partitioned runs no longer say
"Mapped Partition key".

* fixup! UI: Fix partition key display and input handling

* fixup! fixup! UI: Fix partition key display and input handling

* UI: Extract shared partition key normalization util

The trigger form and manual/materialize asset-event flows each
duplicated the same undefined/empty-string-to-null check before
sending partition_key to the API, making the normalization easy to
get subtly wrong in one call site while fixing it in another.

* UI: Simplify partition key visibility check on asset event page

Collapse the redundant undefined/null comparison into a single
loose-equality null check, matching the reviewer's simplification
of the earlier partition key display fix.

* UI: Remove partition-key column auto-visibility from the runs table

The runs table hid the partition_key column by default and revealed it
only for partitioned Dags by fetching Dag details and forcing a table
remount once the answer arrived. That remount-on-load approach was
fragile, and the show/hide behaviour is better solved by a reusable
mechanism (one that could also drive e.g. hiding map_index for
non-mapped tasks), so it is dropped here to be reworked separately.
(cherry picked from commit b767477397dc94b30bb51bd2001196b9392c492e)

Co-authored-by: Wei Lee <[email protected]>

Report URL: https://github.com/apache/airflow/actions/runs/29511352385

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to