Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.7
    • Component/s: core, jcr
    • Labels:
      None

      Description

      As seen on the tck test org.apache.jackrabbit.test.api.query.qom.SelectorTest#testUnknownNodeType

      Currently querying on an unknown node type doesn't fail, I'm not really sure what should be the outcome.

      1. OAK-476.patch
        3 kB
        Alex Parvulescu

        Activity

        Alex Parvulescu created issue -
        Hide
        Alex Parvulescu added a comment -

        attaching proposed patch.

        I'm not really sure this is the best way to solve this problem, as it would always expect some node type definitions to exist, whereas before it would still use the traversing index and return some results.

        As an example of what I'm saying there is one test (failing after applying the patch): org.apache.jackrabbit.oak.api.QueryTest which uses a barebone oak repo with no notyes defined to run a simple query.

        Show
        Alex Parvulescu added a comment - attaching proposed patch. I'm not really sure this is the best way to solve this problem, as it would always expect some node type definitions to exist, whereas before it would still use the traversing index and return some results. As an example of what I'm saying there is one test (failing after applying the patch): org.apache.jackrabbit.oak.api.QueryTest which uses a barebone oak repo with no notyes defined to run a simple query.
        Alex Parvulescu made changes -
        Field Original Value New Value
        Attachment OAK-476.patch [ 12554846 ]
        Hide
        Alex Parvulescu added a comment -

        rev 1441474.

        Show
        Alex Parvulescu added a comment - rev 1441474.
        Alex Parvulescu made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Alex Parvulescu [ alex.parvulescu ]
        Fix Version/s 0.7 [ 12323981 ]
        Resolution Fixed [ 1 ]
        Jukka Zitting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        67d 1h 48m 1 Alex Parvulescu 01/Feb/13 14:59
        Resolved Resolved Closed Closed
        108d 20h 26m 1 Jukka Zitting 21/May/13 12:26

          People

          • Assignee:
            Alex Parvulescu
            Reporter:
            Alex Parvulescu
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development