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

[Data Explorer] Readme #5273

Merged
merged 3 commits into from
Dec 14, 2023

Conversation

ashwin-pc
Copy link
Member

Description

Adds a readme for Data explorer

Issues Resolved

Screenshot

Testing the changes

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
1. Registering the application as a view.
2. Using Data Explorer's state management and data source.
3. Modifying routes to utilize Data Explorer's routes.
4. Adapting the UI to match Data Explorer's panel and workspace components.
Copy link
Member

Choose a reason for hiding this comment

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

should we change workspace to canvas?

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>

### Data Explorer Responsibilities:

1. **Data Source**: Acts as the central point for the data source being explored.
Copy link
Member

Choose a reason for hiding this comment

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

For now it seems to have many data source , but stands for different meanings:

  1. multi datasources: enable OSD to connect to different OS clusters by specifying endpoint url and credentials.
  2. data source: the index pattern or index for retrieving data.

Shall we align on unified terms on this?

Copy link
Member

Choose a reason for hiding this comment

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

+1

I think we could have a terminology reference doc in the repo.

  • Multi-Datasources (OpenSearch Clusters):
    Term: "Connected Clusters" or "Cluster Sources"
    Description: Use this term to refer specifically to the ability of OSD to connect to multiple OpenSearch clusters. Each "Connected Cluster" or "Cluster Source" represents a distinct OpenSearch cluster, identifiable by its endpoint URL and credentials. This feature allows OSD to retrieve and manage data from multiple clusters within the same OSD instance.

  • Data Source (Index Patterns and Indices):
    Term: "Index Patterns"
    Description: This term should be used when referring to the specific index patterns or indices in OpenSearch that are used for data retrieval. "Data Indexes" or "Index Patterns" are the structures or patterns within an individual OpenSearch cluster that OSD queries to fetch and visualize data.

But may be not in this PR. I will approve this PR and we could start discussing on having a terminology reference.

Copy link
Member Author

Choose a reason for hiding this comment

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

Data source here is meant to talk about all data sources, including multi data sources, and even non OpenSearch data sources.

"Data sources: indexes/index patterns" is only for now but it's going to change soon, which is why the naming of the drop downs was changed in anticipation of that.

@ananzh ananzh added backport 2.x docs Improvements or additions to documentation labels Dec 14, 2023
@ananzh ananzh merged commit 3268e37 into opensearch-project:main Dec 14, 2023
10 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Dec 14, 2023
* Data explorer readme
* Update README.md
---------

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
(cherry picked from commit 3268e37)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
manasvinibs pushed a commit that referenced this pull request Dec 14, 2023
* Data explorer readme
* Update README.md
---------


(cherry picked from commit 3268e37)

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x distinguished-contributor docs Improvements or additions to documentation Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants