Accumulo
  1. Accumulo
  2. ACCUMULO-1730

ColumnVisibility parse tree nodes do not have correct location offsets for AND and OR nodes

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.4.0, 1.4.1, 1.4.2, 1.4.3, 1.4.4, 1.5.0
    • Fix Version/s: 1.6.0
    • Component/s: client
    • Labels:
      None

      Description

      Trying to do some transformations on visibility strings and running into issues working with the parse tree:

      Clojure 1.5.1
      user=> (import [org.apache.accumulo.core.security ColumnVisibility])
      org.apache.accumulo.core.security.ColumnVisibility
      user=> (def vis (ColumnVisibility. "(W)|(U|V)"))
      #'user/vis
      user=> (.getTermStart (first (.getChildren (.getParseTree vis))))
      1
      user=> (.getTermEnd (first (.getChildren (.getParseTree vis))))
      2
      user=> (.getTermStart (second (.getChildren (.getParseTree vis))))
      0
      user=> (.getTermEnd (second (.getChildren (.getParseTree vis))))
      8

      Shouldn't those last two be 5 and 8?

        Activity

        John Stoneham created issue -
        Corey J. Nolet made changes -
        Field Original Value New Value
        Fix Version/s 1.4.5 [ 12324754 ]
        Fix Version/s 1.5.1 [ 12324399 ]
        Fix Version/s 1.6.0 [ 12322468 ]
        Eric Newton made changes -
        Summary ColumnVisibility parse tree not always correct ColumnVisibility parse tree nodes do not have correct location offsets for AND and OR nodes
        Assignee Eric Newton [ ecn ]
        Priority Minor [ 4 ] Trivial [ 5 ]
        Eric Newton made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        John Stoneham made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        John Stoneham made changes -
        Status Reopened [ 4 ] Reopened [ 4 ]
        John Stoneham made changes -
        John Stoneham made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Fix Version/s 1.5.1 [ 12324399 ]
        Fix Version/s 1.4.5 [ 12324754 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Eric Newton
            Reporter:
            John Stoneham
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development