ctubbsii commented on PR #5439: URL: https://github.com/apache/accumulo/pull/5439#issuecomment-2770341180
> What made you change your mind on adding `yq` as a dependency? Did I have a previous opinion that was different? I don't remember changing my mind. Perhaps I did. If I did, I think it's because `yq` is a bit more ubiquitous now, and I think that I would prefer avoiding setting a precedent for coupling accumulo-cluster script stuff that should just be for cluster process management (OS-level operations) with Accumulo code (stuff that runs inside the JVM). Also, having recently worked on script stuff, and adding a simple check for the correct version of getopt in accumulo-cluster has given me confidence that we can have the script self-document its dependency in a user-friendly way. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
