Skip to content
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

[DOCFIX] Update bin/alluxio usage format #18128

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

Xenorith
Copy link
Contributor

besides the User-CLI.md doc, update other doc files that refer to bin/alluxio commands

  • remove docs on path config
  • remove starting/stopping job master/worker from contributor docs

Copy link
Contributor

@Kai-Zhang Kai-Zhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall LGTM

@@ -21,8 +21,8 @@ can use in-memory streams, the REST API decouples the stream creation and access
`create` and `open` REST API methods and the `streams` resource endpoints for details).

The HTTP proxy is a standalone server that can be started using
`${ALLUXIO_HOME}/bin/alluxio-start.sh proxy` and stopped using `${ALLUXIO_HOME}/bin/alluxio-stop.sh
proxy`. By default, the REST API is available on port 39999.
`${ALLUXIO_HOME}/bin/alluxio process start proxy` and stopped using `${ALLUXIO_HOME}/bin/alluxio process stop proxy`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

talked to @JiamingMai and i think the proxy is no longer support in 30x.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we still have the code for the process; is there a plan to remove it?
should this doc be completely removed/replaced? @LuQQiu i remember you mentioned the rest api was being replaced; any suggestions on how to handle the doc page?

@Xenorith Xenorith added the type-docs Documentation Defects label Sep 11, 2023
@Xenorith
Copy link
Contributor Author

alluxio-bot, merge this please

@alluxio-bot alluxio-bot merged commit 5ad6e21 into Alluxio:main Sep 11, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-docs Documentation Defects
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants