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

Reply via email to