[ 
https://issues.apache.org/jira/browse/OOZIE-1999?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jozsef Szekrenyes updated OOZIE-1999:
-------------------------------------
    Description: 
The pattern value for IDENTIFIER in the oozie xml-schema is :
"([a-zA-Z_]([\-_a-zA-Z0-9])*){1,39}"
(see 
http://oozie.apache.org/docs/4.0.0/WorkflowFunctionalSpec.html#Oozie_Schema_Version_0.5)

It seems the idea was to limit the length of an identifier to 40 characters. 
Because of the '*', the limitation doesn't have any effect - strings with 
arbitrary length will match the pattern. This caused issues like OOZIE-781.

In order the limit on length to be effective, the '*' should be removed, so the 
pattern should be:
"[a-zA-Z_]([\-_a-zA-Z0-9]){1,39}"

  was:
The pattern value for IDENTIFIER in the oozie xml-schema is :
"([a-zA-Z_]([\-_a-zA-Z0-9])*){1,39}"
(see 
http://oozie.apache.org/docs/4.0.0/WorkflowFunctionalSpec.html#Oozie_Schema_Version_0.5)

It seems the idea was to limit the length of an identifier to 40 characters. 
Because of the '*', the limitation doesn't have any effect - strings with 
arbitrary length will match the pattern. This caused issues like OOZIE-781.

In order the limit on length to be effective, the '*' should be removed, so the 
pattern should be:
"([a-zA-Z_]([\-_a-zA-Z0-9])){1,39}"


> Incorrect pattern regex for IDENTIFIER
> --------------------------------------
>
>                 Key: OOZIE-1999
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1999
>             Project: Oozie
>          Issue Type: Bug
>          Components: coordinator
>            Reporter: Jozsef Szekrenyes
>            Priority: Minor
>
> The pattern value for IDENTIFIER in the oozie xml-schema is :
> "([a-zA-Z_]([\-_a-zA-Z0-9])*){1,39}"
> (see 
> http://oozie.apache.org/docs/4.0.0/WorkflowFunctionalSpec.html#Oozie_Schema_Version_0.5)
> It seems the idea was to limit the length of an identifier to 40 characters. 
> Because of the '*', the limitation doesn't have any effect - strings with 
> arbitrary length will match the pattern. This caused issues like OOZIE-781.
> In order the limit on length to be effective, the '*' should be removed, so 
> the pattern should be:
> "[a-zA-Z_]([\-_a-zA-Z0-9]){1,39}"



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to