gingeekrishna commented on issue #68334: URL: https://github.com/apache/airflow/issues/68334#issuecomment-4821371573
A new PR has been opened to address this issue: #69095 The previous attempt (#68348) was closed after review feedback. This PR addresses all outstanding comments: **Review feedback addressed:** - **[ashb](https://github.com/apache/airflow/pull/68348#pullrequestreview-2926226059)** — asked whether `owner_display_name` is ever set anywhere in the codebase. It is: `action_logging` in `airflow/api_fastapi/logging/decorators.py` writes `owner_display_name=user_display` on every logged API request. The column has existed since the 2.8.0 migration (`0005_2_8_0_add_owner_display_name_to_audit_log_table.py`). This PR exposes the existing value through the read path only. - **[pierrejeambrun](https://github.com/apache/airflow/pull/68348#pullrequestreview-2926017131)** — requested `owner_display_name_pattern` and `owner_display_name_prefix_pattern` filter parameters to match the existing `owner_pattern`/`owner_prefix_pattern` pair. Both are added. - **[Vamsi-klu](https://github.com/apache/airflow/pull/68348#issuecomment-2981869249)** — noted that the UI showed `owner_display_name` but sorting and filtering still targeted `owner`, creating an inconsistency. The new filter and sort parameters operate directly on `Log.owner_display_name`, so searching, sorting, and display are now fully consistent. --- Drafted-by: Claude Code (claude-sonnet-4-6) (no human review before posting) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
