You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@rya.apache.org by pujav65 <gi...@git.apache.org> on 2016/06/23 14:22:13 UTC

[GitHub] incubator-rya pull request #51: Upgrading versions of geomesa

GitHub user pujav65 opened a pull request:

    https://github.com/apache/incubator-rya/pull/51

    Upgrading versions of geomesa

    

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

    $ git pull https://github.com/pujav65/incubator-rya feature/geoUpgrade

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

    https://github.com/apache/incubator-rya/pull/51.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 #51
    
----
commit 05e7c3eed38b27ebee0aee9cb1742134868a0752
Author: pujav65 <pu...@gmail.com>
Date:   2016-06-23T13:26:04Z

    updating to latest geomesa version
    
    updating tests to reflect latest geomesa behavior
    
    ensuring consistent deletes for indexers in addition to core tables
    
    updating to latest geomesa version

commit c2158b17ac748b730faa9c27d466d2ae87194d07
Author: pujav65 <pu...@gmail.com>
Date:   2016-06-23T14:19:39Z

    unignoring test

----


---
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] incubator-rya pull request #51: Upgrading versions of geomesa

Posted by amihalik <gi...@git.apache.org>.
Github user amihalik commented on a diff in the pull request:

    https://github.com/apache/incubator-rya/pull/51#discussion_r68250007
  
    --- Diff: dao/accumulo.rya/src/main/java/mvm/rya/accumulo/AccumuloRyaDAO.java ---
    @@ -202,12 +202,18 @@ public void delete(Iterator<RyaStatement> statements, AccumuloRdfConfiguration c
                     //query first
                     CloseableIteration<RyaStatement, RyaDAOException> query = this.queryEngine.query(stmt, conf);
                     while (query.hasNext()) {
    -                    deleteSingleRyaStatement(query.next());
    +                	RyaStatement resolved = query.next();
    --- End diff --
    
    Nice fix.  Can you fix your whitespace in settings eclispe?  a tab made it in here.


---
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] incubator-rya issue #51: Upgrading versions of geomesa

Posted by pujav65 <gi...@git.apache.org>.
Github user pujav65 commented on the issue:

    https://github.com/apache/incubator-rya/pull/51
  
    will change this to geomesa 1.2.5 or 1.3.x instead.


---
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] incubator-rya issue #51: Upgrading versions of geomesa

Posted by amihalik <gi...@git.apache.org>.
Github user amihalik commented on the issue:

    https://github.com/apache/incubator-rya/pull/51
  
    Please change this to point to the apache master branch.  Also, what's the status of this PR?


---
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] incubator-rya pull request #51: Upgrading versions of geomesa

Posted by pujav65 <gi...@git.apache.org>.
Github user pujav65 closed the pull request at:

    https://github.com/apache/incubator-rya/pull/51


---
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.
---