You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "hussein-awala (via GitHub)" <gi...@apache.org> on 2023/03/11 14:41:25 UTC

[GitHub] [airflow] hussein-awala opened a new pull request, #30043: Disable ordering DagRuns search result by note

hussein-awala opened a new pull request, #30043:
URL: https://github.com/apache/airflow/pull/30043

   closes: #30041
   related: #27849
   
   ---
   After moving the notes to a new table, we removed `note` from the supported search and sort columns for all the APIs, and it seems we forgot disabling this one.  


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] potiuk merged pull request #30043: Disable ordering DagRuns search result by note

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #30043:
URL: https://github.com/apache/airflow/pull/30043


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] uranusjr commented on pull request #30043: Disable ordering DagRuns search result by note

Posted by "uranusjr (via GitHub)" <gi...@apache.org>.
uranusjr commented on PR #30043:
URL: https://github.com/apache/airflow/pull/30043#issuecomment-1465627593

   From a quick look, I think we need to override `apply_order_by` in the data model (our `CustomSQLAInterface`) to support this. That’s probably more work, and I’d argue ordering by note is not particularly essential anyway and can wait for interested contributors.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org