onthebridgetonowhere commented on code in PR #329:
URL: https://github.com/apache/arrow-ballista/pull/329#discussion_r995907358
##########
docker-compose.yml:
##########
@@ -54,8 +54,7 @@ services:
volumes:
- ./benchmarks/data:/data
depends_on:
- ballista-scheduler:
- condition: service_healthy
Review Comment:
> I am tempted to add a `--retry-timeout--seconds 10` option to the
executor, which would fix this. Ballista is targeted at enterprise users, and
they are often not on the latest distro, so would be nice for this to work out
of the box for as many people as possible.
I was thinking of the same option given that the application does not seem
to have a hard constraint to be readily available within milliseconds on
startup. I second the part with having it working out of the box for different
users.
--
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]
