Skip to content

Conversation

amotl
Copy link

@amotl amotl commented Apr 2, 2025

About

pg-mcp uses asyncpg, so it can also be used with CrateDB. A little walkthrough how to use it has been added to the README for CrateDB.

Details

server/resources/schema.py needed to receive a few adjustments to compensate for missing metadata features of CrateDB, nothing serious. @kneth: Maybe carry the gist of that diff forward to the CrateDB issue tracker in one way or another?

@amotl amotl force-pushed the cratedb branch 2 times, most recently from 16d7f61 to 3526db7 Compare April 2, 2025 23:03

# Test disconnect tool if available
break # Exit schema loop once we've found a table
# break # Exit schema loop once we've found a table
Copy link

Choose a reason for hiding this comment

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

Can the break be removed?

Copy link
Author

Choose a reason for hiding this comment

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

It can be removed, sure. I didn't want to change too much compared to the original, that's why it's just a comment.

Copy link
Author

Choose a reason for hiding this comment

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

Please also note this PR is just for visibility purposes. It will probably never be merged, at least not in its current form.

@amotl amotl marked this pull request as draft April 3, 2025 22:00
@amotl amotl force-pushed the cratedb branch 3 times, most recently from 4a8537a to 82733d1 Compare April 5, 2025 21:03
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