[ https://issues.apache.org/jira/browse/TEZ-625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14612984#comment-14612984
]
Cyrille Chépélov edited comment on TEZ-625 at 7/3/15 9:18 AM:
--------------------------------------------------------------
I guess [~cwensel] is MUCH better placed than me here, although [~fs111] is also much better
placed than me as well ;)
Chris, it's about restricting Tez DAG and vertex names to "[:A-Za-z0-9_ -]+$" and 128 characters,
which seems to my untrained eye to encompass what Cascading outputs.
*edit:* I retract the "encompass" statement; to begin with, Cascading outputs the "/" character
in the middle of default DAG names. And as [~fs111] poined out, the end-user may supply anything
right now. Enforcing this policy might break end-user code.
was (Author: cchepelov):
I guess [~cwensel] is MUCH better placed than me here, although [~fs111] is also much better
placed than me as well ;)
Chris, it's about restricting Tez DAG and vertex names to "[:A-Za-z0-9_ -]+$" and 128 characters,
which seems to my untrained eye to encompass what Cascading outputs.
> DAG/Vertex names should be restricted to using [A-Za-z0-9 -_:] and limited to a defined
character limit
> -------------------------------------------------------------------------------------------------------
>
> Key: TEZ-625
> URL: https://issues.apache.org/jira/browse/TEZ-625
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Hitesh Shah
> Assignee: Saikat
> Labels: newbie
> Attachments: TEZ-625.1.patch, TEZ-625.2.patch, TEZ-625.3.patch, TEZ-625.4.patch,
TEZ-625.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
|