accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From milleruntime <...@git.apache.org>
Subject [GitHub] accumulo pull request #178: ACCUMULO-4505: Fix for Shell erroneously reading...
Date Fri, 04 Nov 2016 14:40:39 GMT
GitHub user milleruntime opened a pull request:

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

    ACCUMULO-4505: Fix for Shell erroneously reading accumulo-site.xml

    Fixes in a couple places for ACCUMULO-4505. Shell was getting the site instance config
(which read accumulo-site) when everything it needed was already in ClientConfig. ShellOptionsJC
was ignoring zookeeper properties passed in by the user.  The Initialize class is not executed
when the Shell is run but a static initialization of Zookeeper was forcing a read of accumulo-site
when the class was loaded by getExecutables in Main. 

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

    $ git pull https://github.com/milleruntime/accumulo 4505-1.7

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

    https://github.com/apache/accumulo/pull/178.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 #178
    
----
commit d6622ed75085213a229d168b77302f220dc775a7
Author: milleruntime <mmiller@apache.org>
Date:   2016-11-01T19:17:27Z

    ACCUMULO-4505: Fix for Shell erroneously reading accumulo-site.xml

----


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

Mime
View raw message