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 #147: ACCUMULO-2844 Removed 'slave' terminology
Date Thu, 08 Sep 2016 21:31:01 GMT
Github user joshelser commented on the issue:

    https://github.com/apache/accumulo/pull/147
  
    > >    Do we want to allow fall-back to ease upgrades?
    > -0. Would be slightly helpful in the case of automation. I think w/ the error message
about a missing tservers file, most users will quickly figure out what they need to do. If
we did do it may want to error in the case where tservers and slaves file both exist. In case
where only slaves files exist, could print an error message saying its deprecated and should
be renamed.
    
    This sounds OK to me. We have a bit more leeway with a "real" major release than I'm used
to. I think I would be OK with error'ing out of the script in the case when only the `slaves`
file exists instead of just printing a message (catch it as early as possible and push people
to rename now instead of delaying).


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