accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From joshelser <...@git.apache.org>
Subject [GitHub] accumulo issue #250: ACCUMULO-3384 support custom system props
Date Wed, 19 Apr 2017 19:37:35 GMT
Github user joshelser commented on the issue:

    https://github.com/apache/accumulo/pull/250
  
    > if config is needed system-wide for scan-time components or anything that might be
pluggable which is run before the tables are online.
    
    I can't really argue against the validity of providing some prefix for yet-to-be-defined
server-side extensions (by definitions, those don't yet exist so I can't say if they're a
good idea). Being able to configure such deployed code is a good idea.
    
    However, the original list of "examples" from the ticket ("list contact information for
a system owner or emergency contact, for display on a monitoring site; add notes about last
time a system was manually rebooted or checked for healthy status.") is what bothered me.
It seems like that really isn't the intent of this change -- I'd suggest (like Keith already
did) to make this clear via the Property doc (and user manual).


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