mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-495939110
Thanks @kaxil @zhongjiajie @XD-DENG @mik-laj @Fokko @ashb @r39132 @potiuk
for your advice and help.
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-493015298
@zhongjiajie @XD-DENG @mik-laj @Fokko @ashb @r39132- Just checking if you
have any additional recommendations/comments for this
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-487530657
@XD-DENG and Airflow team - Please let me know if there is anything else
outstanding that I need to address for this PR. Thanks.
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-486320602
@zhongjiajie - I've updated the code based on your feedback. Please let me
know if you have any more advice. Thanks
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-486081315
@mik-laj @XD-DENG @Fokko - Please let me know if you have any additional
advice/comments on this pull request. Thanks
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-484864775
@Fokko, Airflow Team - Please let me know if there is anything else required
for this PR. Thanks.
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-483854332
@zhongjiajie @XD-DENG @mik-laj - Please let me know if you have any more
recommendations for this pull request. Thanks.
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-483055799
@Fokko - I've updated the code based on your feedback. Please let me know
if there is anything else required. Thanks for your
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-481949904
@Fokko @mik-laj @XD-DENG - Please let me know if you have any additional
review recommendations for this PR. Mans
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-479648919
Hi: Just checking if you have any additional review comments/advice. Thanks
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-477355709
@XD-DENG @zhongjiajie - Please let me know if you have more
comments/recommendations and should I resolve your comments. Thanks
mans2singh commented on issue #4967: [AIRFLOW-4147] Operator to publish event
to Redis
URL: https://github.com/apache/airflow/pull/4967#issuecomment-476391674
@XD-DENG - I've updated the test based on your feedback. Please let me know
if you have any additional advice. Thanks
12 matches
Mail list logo
