Details

      Description

      We should define a constant for PATH_SEPARATOR = "/" and use that throughout the code rather than the hardcoded "/". Users can be told to use this constant to be safe in case of future changes.

        Issue Links

          Activity

          Hide
          Mahadev konar added a comment -

          not a blocker. Moving it out of 3.4 release.

          Show
          Mahadev konar added a comment - not a blocker. Moving it out of 3.4 release.
          Hide
          Nitay Joffe added a comment -

          Should also add some constant, say ZNODE_ANY_VERSION, for accepting any version, that is -1.

          Show
          Nitay Joffe added a comment - Should also add some constant, say ZNODE_ANY_VERSION, for accepting any version, that is -1.
          Hide
          Patrick Hunt added a comment -

          +1 - sounds fine to me.

          Note for implementor: name the constant something like ZNODE_PATH_SEPARATOR to ensure that ppl understand that this is znode sep (although that's pretty long).

          Also, if this is done it should be done for both java as well as c client binding. Update docs and tests.

          Show
          Patrick Hunt added a comment - +1 - sounds fine to me. Note for implementor: name the constant something like ZNODE_PATH_SEPARATOR to ensure that ppl understand that this is znode sep (although that's pretty long). Also, if this is done it should be done for both java as well as c client binding. Update docs and tests.

            People

            • Assignee:
              Unassigned
              Reporter:
              Nitay Joffe
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:

                Development