[ 
https://issues.apache.org/jira/browse/CURATOR-499?focusedWorklogId=792401&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-792401
 ]

ASF GitHub Bot logged work on CURATOR-499:
------------------------------------------

                Author: ASF GitHub Bot
            Created on: 19/Jul/22 01:29
            Start Date: 19/Jul/22 01:29
    Worklog Time Spent: 10m 
      Work Description: tisonkun commented on code in PR #300:
URL: https://github.com/apache/curator/pull/300#discussion_r923981812


##########
curator-client/src/test/java/org/apache/curator/utils/TestZKPaths.java:
##########
@@ -26,6 +26,18 @@
 
 public class TestZKPaths
 {
+    @Test

Review Comment:
   @marquiswang lol. Go ahead!





Issue Time Tracking
-------------------

    Worklog Id:     (was: 792401)
    Time Spent: 1h 10m  (was: 1h)

> ZKPaths strips trailing "/" incorrectly when creating sequential nodes
> ----------------------------------------------------------------------
>
>                 Key: CURATOR-499
>                 URL: https://issues.apache.org/jira/browse/CURATOR-499
>             Project: Apache Curator
>          Issue Type: Bug
>            Reporter: Marquis Wang
>            Assignee: Jordan Zimmerman
>            Priority: Minor
>          Time Spent: 1h 10m
>  Remaining Estimate: 0h
>
> I think this was introduced in CURATOR-166, which changed ZKPaths to strip 
> trailing slashes from child nodes.
> This is fine in most cases: it made it so 
> {code:java}
> curatorFramework.create()
>     .forPath("/path/to/node/", data);{code}
> would create a node at {{/path/to/node}}.
> However, if you want to create a sequential node:
> {code:java}
> curatorFramework.create()
>     .withMode(CreateMode.PERSISTENT_SEQUENTIAL)
>     .forPath("/path/to/node/", data);{code}
> In clients after CURATOR-166 (2.7.1 and higher), this will create a node 
> {{/path/to/node00000001}}. Before, and if you call the zookeeper cli with the 
> same options, it would create a node {{/path/to/node/00000001}}.
> This effectively makes it so you cannot use curator to create sequential 
> nodes where the entire node name is the sequence number.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to