From notifications-return-45870-archive-asf-public=cust-asf.ponee.io@accumulo.apache.org Tue Oct 30 23:48:26 2018 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id 1F26E180677 for ; Tue, 30 Oct 2018 23:48:25 +0100 (CET) Received: (qmail 11174 invoked by uid 500); 30 Oct 2018 22:48:15 -0000 Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: jira@apache.org Delivered-To: mailing list notifications@accumulo.apache.org Received: (qmail 11106 invoked by uid 99); 30 Oct 2018 22:48:15 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 30 Oct 2018 22:48:15 +0000 From: GitBox To: notifications@accumulo.apache.org Subject: [GitHub] ctubbsii commented on a change in pull request #126: Created documentation for upgrading to 2.0 Message-ID: <154093969511.10933.3632228288782880465.gitbox@gitbox.apache.org> Date: Tue, 30 Oct 2018 22:48:15 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit ctubbsii commented on a change in pull request #126: Created documentation for upgrading to 2.0 URL: https://github.com/apache/accumulo-website/pull/126#discussion_r229513746 ########## File path: _docs-2/administration/upgrading.md ########## @@ -30,12 +30,18 @@ Below are some changes in 2.0 that you should be aware of: - Tablet server hosts must be listed in a `tservers` file instead of a `slaves` file. To minimize confusion, Accumulo will not start if the old `slaves` file is present. * `accumulo-service` script can be used to start/stop Accumulo services (i.e master, tablet server, monitor) on a single node. - Can be used even if Accumulo was started using `accumulo-cluster` script. -* Accumulo 2.0 logging is configured by three files. Any previous logging customization should be added to these files. +* `accumulo-env.sh` constructs environment variables (such as `JAVA_OPTS` and `CLASSPATH`) used when running Accumulo processes + - This file was used in Accumulo 1.x but has changed signficantly for 2.0 + - Environment variables (such as `$cmd`, `$bin`, `$conf`) are set before `accumulo-env.sh` is loaded and can be used to customize environment. + - The `JAVA_OPTS` variable can be customized to pass command-line arguments to the `java` command that starts Accumulo processes. Review comment: This is good... but in my previous review, I used a specific wording, because it's important to understand that `JAVA_OPTS` is special and distinct from the others. I'm not asking you to use my wording, but I do think it's important to understand that `JAVA_OPTS` is just something we made... and we're explicitly using, whereas things like `CLASSPATH` (and the `MALLOC_*` thing) are not used by us directly, but by other parts of the system. The only thing *we* are specifically asking for (the expected "output" of `accumulo-env.sh`) is `JAVA_OPTS` (although `CLASSPATH` should be set either in the script or before the script is run, otherwise, we won't get very far. I'm not sure this gives enough attention to `JAVA_OPTS`, since it's really the reason why we're running this script (other exported env, or temporary variables aren't required "output" from this script). ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: users@infra.apache.org With regards, Apache Git Services