You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@usergrid.apache.org by snoopdave <gi...@git.apache.org> on 2015/11/05 21:52:42 UTC

[GitHub] usergrid pull request: Query order of operations error

GitHub user snoopdave opened a pull request:

    https://github.com/apache/usergrid/pull/428

    Query order of operations error

    Fix for https://issues.apache.org/jira/browse/USERGRID-1099

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/usergrid usergrid-1099

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/usergrid/pull/428.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #428
    
----
commit c776a2fb34d76f5cd779009a200ab8cf1e288b5d
Author: Dave Johnson <sn...@apache.org>
Date:   2015-11-05T18:58:07Z

    Fix in UnionScanColumn comparator fixes order or operations error https://issues.apache.org/jira/browse/USERGRID-1099

commit d37be7066db054688dc85c3a9b497d51acb07138
Author: Dave Johnson <sn...@apache.org>
Date:   2015-11-05T19:20:36Z

    Merge branch '1.x' into usergrid-1099

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] usergrid pull request: Query order of operations error

Posted by tnine <gi...@git.apache.org>.

Github user tnine commented on the pull request:

    https://github.com/apache/usergrid/pull/428#issuecomment-155862373
  
    +1.  This looks ready to merge


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] usergrid pull request: Query order of operations error

Posted by snoopdave <gi...@git.apache.org>.

Github user snoopdave commented on the pull request:

    https://github.com/apache/usergrid/pull/428#issuecomment-154523717
  
    The existing IntersectionTransitivePagingIT looks good, but it tests a more simple query, one that intersection two slice iterators -- so it did not tickle this bug.  
    
    I think we are ready to merge this into 1.x. If we want more complex query tests we should open up a new JIRA for that. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] usergrid pull request: Query order of operations error

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/usergrid/pull/428


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---