-
Notifications
You must be signed in to change notification settings - Fork 196
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update root README.md
and other documentation with latest changes
#1113
base: main
Are you sure you want to change the base?
Conversation
and cleanup some of the context. Relates to apache#1105
c7bb151
to
e7ccbb1
Compare
README.md
adding example how to use ...README.md
and other documentation with latest changes
1d78676
to
315b775
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we still have a rest implementation that works? I was testing out the other day and couln't get a response from the REST endpoint. I may have misconfigured the cluster on AWS, though. Was able to run jobs against it, just not use the REST API.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, it's a optional feature (rest-api
) of scheduler crate and its disabled by default. should work when enabled, otherwise its a bug and should be fixed
thanks for your review @tbar4 |
docs/source/contributors-guide/ballista_architecture.excalidraw.svg
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a couple of comments, but LGTM.
351930f
to
9797927
Compare
and cleanup some of the context.
Which issue does this PR close?
Closes #1105.
Rationale for this change
What changes are included in this PR?
Are there any user-facing changes?