You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/09/23 16:40:02 UTC

[GitHub] [airflow] rjmcginness opened a new pull request, #26636: Fix xcom arg.py .zip bug

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

   Modified xcom_arg.py to test against ArgNotSet type, rather than an instance (NOTSET).
   Fixes the filling of the _ZipResult with an erroneous fill value, when .zip is called with the
   default fillvalue=NOTSET.
   
   closes: #26499
   related: #26499
   


-- 
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] rjmcginness commented on pull request #26636: Fix xcom arg.py .zip bug

Posted by GitBox <gi...@apache.org>.
rjmcginness commented on PR #26636:
URL: https://github.com/apache/airflow/pull/26636#issuecomment-1257412929

   Sounds good. Glad to help. Airflow is a great project. i’m hoping to learn and do more here. 


-- 
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 #26636: Fix xcom arg.py .zip bug

Posted by GitBox <gi...@apache.org>.
potiuk merged PR #26636:
URL: https://github.com/apache/airflow/pull/26636


-- 
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 #26636: Fix xcom arg.py .zip bug

Posted by GitBox <gi...@apache.org>.
uranusjr commented on PR #26636:
URL: https://github.com/apache/airflow/pull/26636#issuecomment-1257411804

   Thanks a lot! Since the apparent issue would be fixed with this PR, the specific root cause can be looked into without breaking things, so there’s no hurry, feel free to do it when you have time.


-- 
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 commented on pull request #26636: Fix xcom arg.py .zip bug

Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #26636:
URL: https://github.com/apache/airflow/pull/26636#issuecomment-1257720061

   > Sounds good. Glad to help. Airflow is a great project. i’m hoping to learn and do more here.
   
   :eyes: 


-- 
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] boring-cyborg[bot] commented on pull request #26636: Fix xcom arg.py .zip bug

Posted by GitBox <gi...@apache.org>.
boring-cyborg[bot] commented on PR #26636:
URL: https://github.com/apache/airflow/pull/26636#issuecomment-1257721126

   Awesome work, congrats on your first merged pull request!
   


-- 
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] rjmcginness commented on pull request #26636: Fix xcom arg.py .zip bug

Posted by GitBox <gi...@apache.org>.
rjmcginness commented on PR #26636:
URL: https://github.com/apache/airflow/pull/26636#issuecomment-1257410402

   Sure. I would be happy to do that. End of the day here and work tomorrow, so it may take a little while. However, I will work on it. 


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