[
https://issues.apache.org/jira/browse/BEAM-12469?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17549380#comment-17549380
]
Danny McCormick commented on BEAM-12469:
----------------------------------------
This issue has been migrated to https://github.com/apache/beam/issues/20982
> Replace _unified_repr with repr
> -------------------------------
>
> Key: BEAM-12469
> URL: https://issues.apache.org/jira/browse/BEAM-12469
> Project: Beam
> Issue Type: Improvement
> Components: sdk-py-core
> Reporter: Udi Meiri
> Priority: P3
> Labels: starter
>
> 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.
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
