You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@accumulo.apache.org by keith-turner <gi...@git.apache.org> on 2015/11/20 05:48:50 UTC

[GitHub] accumulo pull request: ACCUMULO-2883 Added locating tablets to API

GitHub user keith-turner opened a pull request:

    https://github.com/apache/accumulo/pull/53

    ACCUMULO-2883 Added locating tablets to API

    I have modified Gora to use this update in a [branch][2].  I plan to experiment with modifying [geowave][1] to use this new API before pusing to master.
    
    [1]: https://github.com/ngageoint/geowave
    [2]: https://github.com/keith-turner/gora/tree/GORA-414-180

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

    $ git pull https://github.com/keith-turner/accumulo ACCUMULO-2883

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

    https://github.com/apache/accumulo/pull/53.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 #53
    
----
commit f89e386148173dbd902765da711c0c553fd37fa6
Author: Keith Turner <kt...@apache.org>
Date:   2015-11-13T04:31:22Z

    ACCUMULO-2883 Added locating tablets to API

----


---
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] accumulo pull request: ACCUMULO-2883 Added locating tablets to API

Posted by keith-turner <gi...@git.apache.org>.
Github user keith-turner commented on the pull request:

    https://github.com/apache/accumulo/pull/53#issuecomment-162987572
  
    I was able to get geowave working with these changes ( [my branch](https://github.com/keith-turner/geowave/tree/use_locator_api)). Was able to successfully run the geowave test BasicMapReduceIT that @rfecher told me about in the geowave chat room.
    
    I'm happy with this PR and will push soon.


---
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] accumulo pull request: ACCUMULO-2883 Added locating tablets to API

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

    https://github.com/apache/accumulo/pull/53


---
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] accumulo pull request: ACCUMULO-2883 Added locating tablets to API

Posted by joshelser <gi...@git.apache.org>.
Github user joshelser commented on the pull request:

    https://github.com/apache/accumulo/pull/53#issuecomment-163013460
  
    :+1: Excellent! Great work, Keith.


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