tez-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Cyrille Chépélov (JIRA) <j...@apache.org>
Subject [jira] [Comment Edited] (TEZ-625) DAG/Vertex names should be restricted to using [A-Za-z0-9 -_:] and limited to a defined character limit
Date Fri, 03 Jul 2015 09:19:05 GMT

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

Mime
View raw message