Details

      Description

      the TCK query test SetOffsetTest fails in the setup jcr2spi - spi2dav(ex) - jcr-server.
      not sure whether it is due to missing implementation on client or server part of something completely different....

        Activity

        Hide
        Christian Stocker added a comment -

        Some more info about what's the actual issue is here JCR-2884

        Show
        Christian Stocker added a comment - Some more info about what's the actual issue is here JCR-2884
        Hide
        Christian Stocker added a comment -

        Further investigation showed me, that the requiredNamespace is null, but node.getNamespaceURI() is "", can we assume, that "" == null in XML namespace speak? That should fix the problem then.

        Show
        Christian Stocker added a comment - Further investigation showed me, that the requiredNamespace is null, but node.getNamespaceURI() is "", can we assume, that "" == null in XML namespace speak? That should fix the problem then.
        Hide
        Christian Stocker added a comment -

        Attached is a simple patch, which solves the issue.

        Show
        Christian Stocker added a comment - Attached is a simple patch, which solves the issue.
        Hide
        angela added a comment -

        the TODO you mention in the JCR-2884 means that the code isn't completed yet and that we in fact
        should use a namespace.

        the patch you added isn't a proper fix for the half-way implementation of the query offset.
        could you please file a separate issue for your findings about the namespace mismatch? thanks

        Show
        angela added a comment - the TODO you mention in the JCR-2884 means that the code isn't completed yet and that we in fact should use a namespace. the patch you added isn't a proper fix for the half-way implementation of the query offset. could you please file a separate issue for your findings about the namespace mismatch? thanks
        Hide
        Christian Stocker added a comment -

        The new issue is at JCR-2888 with the same patch attached.

        Show
        Christian Stocker added a comment - The new issue is at JCR-2888 with the same patch attached.
        Hide
        Lukas Kahwe Smith added a comment -

        so is this issue resolved now?

        Show
        Lukas Kahwe Smith added a comment - so is this issue resolved now?
        Hide
        Jukka Zitting added a comment -

        The test indeed passes now, so in revision 1226750 I removed them from the known.issues exclusion list. Resolving as fixed.

        Show
        Jukka Zitting added a comment - The test indeed passes now, so in revision 1226750 I removed them from the known.issues exclusion list. Resolving as fixed.

          People

          • Assignee:
            Jukka Zitting
            Reporter:
            angela
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development