[GitHub] [beam] kennknowles opened a new issue, #18633: typehints.TypeVariable issues with __hash__

[GitBox <gi...@apache.org>]

kennknowles opened a new issue, #18633:
URL: https://github.com/apache/beam/issues/18633

   This class currently does not define a __hash__ function. This defies the assumption made by sets and dicts which contain TypeConstraint objects.
   
   In Python 3, classes which define ___eq___ also need to explicitly define ___hash___ to be hashable. However, this causes _combiners_test.CombineTest.test_to_list_and_to_dict_ to throw an error in slow_stream.py (adding NoneType and long).
   
   There are multiple TypeVariable instances with name 'K' or 'V' throughout the codebase. The equality of these instances may possibly cause issues.
   
    
   
   Imported from Jira [BEAM-3730](https://issues.apache.org/jira/browse/BEAM-3730). Original Jira may contain additional context.
   Reported by: luke-zhu.


-- 
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: github-unsubscribe@beam.apache.org.apache.org

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