You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/03/21 07:16:34 UTC

[GitHub] [accumulo] ctubbsii commented on issue #1568: Support multiple tservers / node in accumulo-service

ctubbsii commented on issue #1568: Support multiple tservers / node in accumulo-service
URL: https://github.com/apache/accumulo/pull/1568#issuecomment-602006769
 
 
   > Acknowledged, thank you Christopher for your candid feedback. I'll think through the alternatives.
   
   No problem. Sorry if I seem overly critical. I don't intend to be harsh.
   
   Honestly, I think you could probably adapt what you've already done into the `accumulo-cluster` script pretty easily instead, and just export the environment variable before each call to the `accumulo-service` script in there. The `accumulo-service` script itself would need only minor updates to leverage the environment variable for its own output file names.
   
   > Also appreciate the tip about the commit message, that mistake will not be repeated.
   
   No worries. (FWIW, one could legitimately argue that I only routinely provide that advice to keep the record for longest/worst one-line git log message in 8851c3578e5b12281a47fdb0b0f7c43516a544d6 :smiley_cat:)

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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