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

Refine kyuubi extension config docs #638

Closed
wants to merge 1 commit into from
Closed

Refine kyuubi extension config docs #638

wants to merge 1 commit into from

Conversation

ulysses-you
Copy link
Contributor

Why are the changes needed?

Refine docs.

How was this patch tested?

Not needed.

@ulysses-you ulysses-you added this to the v1.2.0 milestone May 14, 2021
@ulysses-you ulysses-you self-assigned this May 14, 2021
ulysses-you added a commit that referenced this pull request May 14, 2021
<!--
Thanks for sending a pull request!

Here are some tips for you:
  1. If this is your first time, please read our contributor guidelines: https://kyuubi.readthedocs.io/en/latest/community/contributions.html
  2. If the PR is related to an issue in https://github.com/NetEase/kyuubi/issues, add '[KYUUBI #XXXX]' in your PR title, e.g., '[KYUUBI #XXXX] Your PR title ...'.
  3. If the PR is unfinished, add '[WIP]' in your PR title, e.g., '[WIP][KYUUBI #XXXX] Your PR title ...'.
-->

### _Why are the changes needed?_
<!--
Please clarify why the changes are needed. For instance,
  1. If you add a feature, you can talk about the use case of it.
  2. If you fix a bug, you can clarify why it is a bug.
-->
Refine docs.

### _How was this patch tested?_
Not needed.

Closes #638 from ulysses-you/refine-config-docs.

Closes #638

e9f3f37 [ulysses-you] refine

Authored-by: ulysses-you <ulyssesyou18@gmail.com>
Signed-off-by: ulysses-you <ulyssesyou18@gmail.com>
(cherry picked from commit ccb891d)
Signed-off-by: ulysses-you <ulyssesyou18@gmail.com>
@ulysses-you ulysses-you deleted the refine-config-docs branch May 14, 2021 03:35
@ulysses-you
Copy link
Contributor Author

thanks, mreging to master/v1.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants