[PR] Unify collection.abc imports [airflow]

["uranusjr (via GitHub)" <gi...@apache.org>]

uranusjr opened a new pull request, #57914:
URL: https://github.com/apache/airflow/pull/57914

   Some import inconsistencies noticed when working on #57878


-- 
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

Re: [PR] Unify collection.abc imports [airflow]

["github-actions[bot] (via GitHub)" <gi...@apache.org>]

github-actions[bot] commented on PR #57914:
URL: https://github.com/apache/airflow/pull/57914#issuecomment-3494841274

   ### Backport failed to create: v3-1-test. View the failure log <a href='https://github.com/apache/airflow/actions/runs/19124274893'> Run details </a>
   
   <table>
       <tr>
           <th>Status</th>
           <th>Branch</th>
           <th>Result</th>
       </tr>
       <tr>
           <td>❌</td>
           <td>v3-1-test</td>
           <td><a href="https://github.com/apache/airflow/commit/15b683284722c179c57998947999bd4b5497d184"><img src='https://img.shields.io/badge/Commit-15b6832-red' alt='Commit Link'></a></td>
       </tr>
   </table>
   
   You can attempt to backport this manually by running:
   
   ```bash
   cherry_picker 15b6832 v3-1-test
   ```
   
   This should apply the commit to the v3-1-test branch and leave the commit in conflict state marking
   the files that need manual conflict resolution.
   
   After you have resolved the conflicts, you can continue the backport process by running:
   
   ```bash
   cherry_picker --continue
   ```
   


-- 
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

Re: [PR] Unify collection.abc imports [airflow]

["uranusjr (via GitHub)" <gi...@apache.org>]

uranusjr commented on code in PR #57914:
URL: https://github.com/apache/airflow/pull/57914#discussion_r2496840176


##########
airflow-core/src/airflow/api_fastapi/core_api/datamodels/connections.py:
##########
@@ -121,7 +120,7 @@ class ConnectionHookMetaData(BaseModel):
     default_conn_name: str | None
     hook_name: str
     standard_fields: StandardHookFields | None
-    extra_fields: abc.MutableMapping | None
+    extra_fields: Mapping | None

Review Comment:
   This is only ever used in the front end so there’s no point it being mutable.



-- 
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

Re: [PR] Unify collection.abc imports [airflow]

["uranusjr (via GitHub)" <gi...@apache.org>]

uranusjr merged PR #57914:
URL: https://github.com/apache/airflow/pull/57914


-- 
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