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:06:39 UTC
[GitHub] [beam] damccorm opened a new issue, #20907: Document: _namespace is a reserved field for DoFns (used for display data)
damccorm opened a new issue, #20907:
URL: https://github.com/apache/beam/issues/20907
User reported issue when _namespace is not used for display data:
\> self._populate_items(display_data_dict)
\> File third_party/py/apache_beam/transforms/display.py", line 117, in _populate_items
\> element._namespace(), element.display_data())
\> TypeError: 'str' object is not callable
Imported from Jira [BEAM-12340](https://issues.apache.org/jira/browse/BEAM-12340). Original Jira may contain additional context.
Reported by: altay.
--
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] tvalentyn closed issue #20907: Document: _namespace is a reserved field for DoFns (used for display data)
Posted by "tvalentyn (via GitHub)" <gi...@apache.org>.
tvalentyn closed issue #20907: Document: _namespace is a reserved field for DoFns (used for display data)
URL: https://github.com/apache/beam/issues/20907
--
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] svetakvsundhar commented on issue #20907: Document: _namespace is a reserved field for DoFns (used for display data)
Posted by "svetakvsundhar (via GitHub)" <gi...@apache.org>.
svetakvsundhar commented on issue #20907:
URL: https://github.com/apache/beam/issues/20907#issuecomment-1513460034
@tvalentyn ; could you provide more context on this issue? Looks like we should document this here [1] if IIUC?
[1]
https://github.com/apache/beam/blob/master/website/www/site/content/en/documentation/transforms/python/elementwise/pardo.md
--
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 commented on issue #20907: Document: _namespace is a reserved field for DoFns (used for display data)
Posted by "tvalentyn (via GitHub)" <gi...@apache.org>.
tvalentyn commented on issue #20907:
URL: https://github.com/apache/beam/issues/20907#issuecomment-1514617538
I think we can rename HasDisplayData._namespace() to HasDisplayData._get_display_data_namespace() to prevent such collisions. As far as I can tell, the changes will be contained to `display.py`.
--
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