accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eric Newton (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-1730) ColumnVisibility parse tree nodes do not have correct location offsets for AND and OR nodes
Date Wed, 16 Oct 2013 01:49:42 GMT

    [ https://issues.apache.org/jira/browse/ACCUMULO-1730?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13796317#comment-13796317
] 

Eric Newton commented on ACCUMULO-1730:
---------------------------------------

For some reason the patch does not apply clean:

{noformat}
$ git apply ~/Downloads/0001-ACCUMULO-1730-Reinsert-changes-inadvertently-lost-in.patch 
error: patch failed: core/src/main/java/org/apache/accumulo/core/security/ColumnVisibility.java:302
error: core/src/main/java/org/apache/accumulo/core/security/ColumnVisibility.java: patch does
not apply
error: patch failed: core/src/test/java/org/apache/accumulo/core/security/ColumnVisibilityTest.java:152
error: core/src/test/java/org/apache/accumulo/core/security/ColumnVisibilityTest.java: patch
does not apply
{noformat}


> ColumnVisibility parse tree nodes do not have correct location offsets for AND and OR
nodes
> -------------------------------------------------------------------------------------------
>
>                 Key: ACCUMULO-1730
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1730
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.0, 1.4.1, 1.4.2, 1.4.3, 1.4.4, 1.5.0
>            Reporter: John Stoneham
>            Assignee: Eric Newton
>            Priority: Trivial
>             Fix For: 1.4.5, 1.5.1, 1.6.0
>
>         Attachments: 0001-ACCUMULO-1730-Reinsert-changes-inadvertently-lost-in.patch
>
>
> 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?



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message