Github user ctubbsii commented on a diff in the pull request:

    https://github.com/apache/accumulo/pull/218#discussion_r101363077
  
    --- Diff: INSTALL.md ---
    @@ -40,13 +40,13 @@ Accumulo has some optional native code that improves 
its performance and
     stability. Before configuring Accumulo, attempt to build this native code
     with the following command.
     
    -    accumulo build-native
    +    ./contrib/accumulo-native
    --- End diff --
    
    I'm not sure contrib is the best place for this. I think of contrib items 
as stuff that adds on to, but is unknown to, the core project. The native 
libraries are something that are optional, but known to Accumulo, and tightly 
coupled with it. I think of contrib stuff as optional stuff many users can 
ignore. But, native libraries are something we strongly recommend.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to