dlmarion commented on PR #5439: URL: https://github.com/apache/accumulo/pull/5439#issuecomment-2770466113
> 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). [Here](https://github.com/apache/accumulo/issues/2138#issuecomment-866979795), but it sounds like the reason for your objection back then has been resolved now. `jq` with json was also proposed, which met your criteria for packaging. Well, at least we have some examples of how to use yq in the linked comment. I'm sure the structure has changed since then, so they likely need modification. -- 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: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
