austin3dickey commented on PR #41031: URL: https://github.com/apache/arrow/pull/41031#issuecomment-2040071984
>We have a few other places that use the same shebang and I think we should update them as well. Just looked into this briefly and found places in archery that might fail if we remove the shebangs: https://github.com/apache/arrow/blob/83359d6958273be534f376f40976196c14675c8b/dev/archery/archery/integration/tester_js.py#L27-L30 So we could go through and fix all these, but it might be more trouble than it's worth. Would you mind if I keep this PR scoped to just fixing the benchmarks' portability? (For context, I'm trying to move the orchestration of the Arrow merge-commit JS benchmarks off a machine in someone's basement to an EC2 instance. This PR unblocks that work.) -- 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]
