[GitHub] [airflow] Ritika-Singhal commented on issue #21348: Status of testing Providers that were prepared on February 05, 2022

2022-02-05 Thread GitBox


Ritika-Singhal commented on issue #21348:
URL: https://github.com/apache/airflow/issues/21348#issuecomment-1030739093


   For an additional fix for #19787, I have created another pull request #21353


-- 
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-unsubscr...@airflow.apache.org

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




[GitHub] [airflow] Ritika-Singhal commented on issue #21348: Status of testing Providers that were prepared on February 05, 2022

2022-02-05 Thread GitBox


Ritika-Singhal commented on issue #21348:
URL: https://github.com/apache/airflow/issues/21348#issuecomment-1030723900


   #19787 requires one additional fix in here: 
   
https://github.com/apache/airflow/blob/providers-amazon/3.0.0rc1/airflow/providers/amazon/aws/operators/glue.py
 
   Need to modify the default value of `num_of_dpus` to `None` (which is 
currently set to 6). 
   
   Without this fix, it becomes inconsistent with the logic changed in the 
`[airflow/providers/amazon/aws/hooks/glue.py](https://github.com/apache/airflow/pull/19787/files#diff-039a9f654ab15c81177e8747d7dd210768a6c3dcb5404be66f4e3c3a7b732c6a)`.
 It then gives the error to the user if user doesn't specify `num_of_dpus=None`
   


-- 
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-unsubscr...@airflow.apache.org

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