markrmiller commented on pull request #365: URL: https://github.com/apache/lucene/pull/365#issuecomment-939453036
Yeah that may be the way to go after all. I've always rejected it I guess from mostly an impractical snobbish micro reaction to the idea. But maybe the less likely to surprise quality wins. It was pretty surprising this issue hid out for so long. I've run it on 5 towers, a mini desktop box, a couple raspberry pi's, across Ubuntu, Arch/Manjaro, x86, arm, bash, zsh, a couple others have ran it. Shell scripts ... Until now that was down to only bothering me around non functional nits. There is one more functional nit left Id love to lose. Profilers can take options like: -prof async:alloc;output=flamegraph We have to escape those semi colons or risk incorrect parsing if the cmd line. That's annoying. Kafka uses this script approach as well, (but with an Uber jar), and they just document it. I think it's difficult to deal with, but I'd love to escape that. -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
