Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Update out-of-date documentation #820

Merged
merged 9 commits into from
Nov 18, 2020

Conversation

dai-chen
Copy link
Member

@dai-chen dai-chen commented Nov 12, 2020

Issue #, if available: #528, #607

Description of changes:

  1. Update out-of-date content (JDK version, project structure, Gradle commands etc) in
    1.1 README: https://github.com/dai-chen/sql/tree/update-docs#documentation
    1.2 Dev Guide: https://github.com/dai-chen/sql/blob/update-docs/docs/developing.rst#project-strucure
  2. Add a experimental section for new query engine in README: https://github.com/dai-chen/sql/tree/update-docs#experimental
  3. Clarify projects field is nested in JSON Support doc: https://github.com/dai-chen/sql/blob/update-docs/docs/user/beyond/partiql.rst#example-employees
  4. Add metrics doc for PPL: https://github.com/dai-chen/sql/blob/update-docs/docs/experiment/ppl/admin/monitoring.rst

TODO: Will push more design docs once new engine is ready to be enabled.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@dai-chen dai-chen marked this pull request as ready for review November 17, 2020 21:56
@dai-chen dai-chen requested review from penghuo and chloe-zh November 17, 2020 21:56
Copy link
Contributor

@penghuo penghuo left a comment

Choose a reason for hiding this comment

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

Thanks for the change!

@dai-chen dai-chen merged commit 815f4ad into opendistro-for-elasticsearch:develop Nov 18, 2020
@dai-chen dai-chen deleted the update-docs branch November 18, 2020 22:07
penghuo pushed a commit that referenced this pull request Dec 15, 2020
* Update readme and dev guide

* Update partiql doc for nested field

* Add PPL metrics

* Add PPL metrics

* Add PPL metrics

* Add PPL metrics

* Prepare PR
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants