Skip to content

Update data main#1059

Merged
chowbao merged 8 commits into
mainfrom
update-data-main
Oct 28, 2024
Merged

Update data main#1059
chowbao merged 8 commits into
mainfrom
update-data-main

Conversation

@chowbao
Copy link
Copy Markdown
Contributor

@chowbao chowbao commented Oct 25, 2024

Add data service chart to data main section

Comment thread docs/data/README.mdx Outdated
@stellar-jenkins
Copy link
Copy Markdown

Comment thread docs/data/README.mdx Outdated
Comment thread docs/data/README.mdx Outdated
Copy link
Copy Markdown
Contributor

@sydneynotthecity sydneynotthecity left a comment

Choose a reason for hiding this comment

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

I like the table comparison, I think we need to give the user a little more details on what each of the services is before comparing capabilities

Comment thread docs/data/README.mdx Outdated
Comment thread docs/data/README.mdx Outdated
Comment thread docs/data/README.mdx Outdated
Comment thread docs/data/README.mdx Outdated
Comment thread docs/data/README.mdx Outdated
Copy link
Copy Markdown
Contributor

@sydneynotthecity sydneynotthecity left a comment

Choose a reason for hiding this comment

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

couple nits, but looks good enough to merge imo 🙌

Comment thread docs/data/README.mdx Outdated
- **RPC** - live network gateway
- **Horizon** - API for network state data
- **Galexie** - exports raw ledger metadata files
- **Hubble** - OLAP database for network data
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

dumb question: Do you think enough of the readership will understand OLAP? If not, we may want to be more explicit and say either 'modern data warehouse' or 'analytics database'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good point. I like "analytics database"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated in e250dd6

Comment thread docs/data/README.mdx Outdated
| Stellar Asset Contracts | ✅ | ✅ | ✅ | ✅ |
| API | ✅ | ✅ | ❌ | ❌ |
| Transaction Submission | ✅ | ✅ | ❌ | ❌ |
| Curated and Parsed Data | ✅ | ✅ | ❌ | ✅ |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

oooh maybe unpopular opinion, but I would put ❌ for RPC for curated/parsed data

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ooo that is unpopular. I'm fine with doing that just for the sake of differentiating RPC and Horizon more

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated in e250dd6

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Idt that's unpopular

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

:shocked-pikachu-face:

@stellar-jenkins
Copy link
Copy Markdown

Comment thread docs/data/README.mdx Outdated
Comment thread docs/data/README.mdx Outdated
Comment thread docs/data/README.mdx Outdated
Comment thread docs/data/README.mdx Outdated
Comment thread docs/data/README.mdx
Co-authored-by: Molly Karcher <molly@stellar.org>
@stellar-jenkins
Copy link
Copy Markdown

@stellar-jenkins
Copy link
Copy Markdown

@stellar-jenkins
Copy link
Copy Markdown

Something went wrong with PR preview build please check

@chowbao chowbao merged commit 753be56 into main Oct 28, 2024
@chowbao chowbao deleted the update-data-main branch October 28, 2024 18:43
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.

5 participants