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/04/21 10:51:33 UTC

[GitHub] [airflow] uranusjr commented on a diff in pull request #23136: Add special exception for "host field is not hashable"

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


##########
airflow/providers/elasticsearch/log/es_task_handler.py:
##########
@@ -22,7 +22,7 @@
 from datetime import datetime
 from operator import attrgetter
 from time import time
-from typing import List, Optional, Tuple, Union
+from typing import Hashable, List, Optional, Tuple, Union

Review Comment:
   Use `collections.abc.Hashable` instead (I believe that’s the canonical import)



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