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

feat: postgres_fdw demo #66

Merged
merged 2 commits into from
Dec 13, 2024
Merged

feat: postgres_fdw demo #66

merged 2 commits into from
Dec 13, 2024

Conversation

sunng87
Copy link
Member

@sunng87 sunng87 commented Dec 13, 2024

This demo shows how GreptimeDB works with postgres_fdw to provide foreign data access for Postgres. It follows our best practice for demo that: 1. works with both local and cloud GreptimeDB, 2. (almost) zero dependency required to setup.

@sunng87 sunng87 requested a review from waynexia December 13, 2024 07:56
@waynexia waynexia requested a review from Copilot December 13, 2024 08:17

Choose a reason for hiding this comment

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

Copilot reviewed 3 out of 6 changed files in this pull request and generated no comments.

Files not reviewed (3)
  • postgres-fdw/.gitignore: Language not supported
  • postgres-fdw/docker-entrypoint-initdb.d/init.sh: Language not supported
  • postgres-fdw/greptime.env.sample: Language not supported
Comments suppressed due to low confidence (1)

postgres-fdw/vector.toml:12

  • [nitpick] The sink name 'logs_out2' is ambiguous and does not clearly describe its purpose. Consider renaming it to something more descriptive, such as 'greptimedb_sink'.
[sinks.logs_out2]
Copy link
Member

@waynexia waynexia left a comment

Choose a reason for hiding this comment

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

Pretty cool

Comment on lines +38 to +40
Next, use `psql` to access vanilla postgres using `psql -h 127.0.0.1 -p 5432 -U
postgres`. You can check foreign servers and tables, query data from the remote
GreptimeDB.
Copy link
Member

Choose a reason for hiding this comment

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

Do we need to explain briefly how PostgreSQL recognizes GreptimeDB? Maybe a link to init.sh

Copy link
Member Author

Choose a reason for hiding this comment

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

It's explained in "How it works" section below.

@sunng87 sunng87 merged commit 3cd0135 into main Dec 13, 2024
1 check passed
@sunng87 sunng87 deleted the demo/postgres-fdw branch December 13, 2024 08:30
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.

2 participants