You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/06/04 20:50:43 UTC

[GitHub] [beam] damccorm opened a new issue, #20982: Replace _unified_repr with repr

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

   function: https://github.com/apache/beam/blob/8926c43f20ab19b8e445d519cf424e881365fc6c/sdks/python/apache_beam/typehints/typehints.py#L354
   
   \_unified_repr was written for Python 2.x, but repr() should now be a suitable replacement (see https://www.python.org/dev/peps/pep-3155/).
   
   repr() is typically more verbose than \_\_name\_\_ unfortunately (<class 'int'\> vs int), breaking lots of unit tests that depend on exact strings. These test could also be updated to use the less brittle self.assertRaisesRegex.
   
   Imported from Jira [BEAM-12469](https://issues.apache.org/jira/browse/BEAM-12469). Original Jira may contain additional context.
   Reported by: udim.


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

[GitHub] [beam] hurutoriya commented on issue #20982: Replace _unified_repr with repr

Posted by GitBox <gi...@apache.org>.

hurutoriya commented on issue #20982:
URL: https://github.com/apache/beam/issues/20982#issuecomment-1220781615

   .take-issue


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

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

[GitHub] [beam] tvalentyn closed issue #20982: Replace _unified_repr with repr

Posted by GitBox <gi...@apache.org>.

tvalentyn closed issue #20982: Replace _unified_repr with repr
URL: https://github.com/apache/beam/issues/20982


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

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