[jira] Commented: (ZOOKEEPER-348) Creating node with path ending in / with sequence flag set

[Hadoop QA (JIRA)]

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12695619#action_12695619
 ] 

Hadoop QA commented on ZOOKEEPER-348:
-

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org
  against trunk revision 761816.

+1 @author.  The patch does not contain any @author tags.

-1 tests included.  The patch doesn't appear to include any new or modified 
tests.
Please justify why no tests are needed for this patch.

-1 patch.  The patch command could not apply the patch.

Console output: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-vesta.apache.org/16/console

This message is automatically generated.

 Creating node with path ending in / with sequence flag set
 

 Key: ZOOKEEPER-348
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-348
 Project: Zookeeper
  Issue Type: Bug
  Components: c client
Affects Versions: 3.1.0, 3.1.1
Reporter: Jeff Terrace
Assignee: Patrick Hunt
Priority: Minor
 Fix For: 3.2.0


 In 3.0.1, I could create a sequence node like this:
 /nodes/001
 like this:
 string path = /nodes/;
 string value = data;
 int rc = zoo_acreate(zh, path.c_str(), value.c_str(), value.length(), 
 ZOO_OPEN_ACL_UNSAFE, ZOO_EPHEMERAL | ZOO_SEQUENCE, czoo_created, where);
 In 3.1.1, this fails with error -8 (ZBADARGUMENTS).
 Adding something after the / in the path makes the code work fine:
 string path = /nodes/n;
 I assume something is checking if the path ends in / but not checking the 
 sequence flag.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Commented: (ZOOKEEPER-348) Creating node with path ending in / with sequence flag set

[Mahadev konar (JIRA)]

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12689741#action_12689741
 ] 

Mahadev konar commented on ZOOKEEPER-348:
-

it just came to my mind that we put validating path at the client side in  
ZOOKEEPER-16. can you try using 3.1* versions? you should get errors in both 
the cases in 3.1* versions.


 Creating node with path ending in / with sequence flag set
 

 Key: ZOOKEEPER-348
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-348
 Project: Zookeeper
  Issue Type: Bug
  Components: c client
Affects Versions: 3.1.1
Reporter: Jeff Terrace
Priority: Minor
 Fix For: 3.2.0


 In 3.0.1, I could create a sequence node like this:
 /nodes/001
 like this:
 string path = /nodes/;
 string value = data;
 int rc = zoo_acreate(zh, path.c_str(), value.c_str(), value.length(), 
 ZOO_OPEN_ACL_UNSAFE, ZOO_EPHEMERAL | ZOO_SEQUENCE, czoo_created, where);
 In 3.1.1, this fails with error -8 (ZBADARGUMENTS).
 Adding something after the / in the path makes the code work fine:
 string path = /nodes/n;
 I assume something is checking if the path ends in / but not checking the 
 sequence flag.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.