Skip to content

doc: Add a readme for the SQL catalog #1459

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

kyteware
Copy link
Contributor

Which issue does this PR close?

What changes are included in this PR?

A README.md for the SQL catalog. Includes information on including sqlite database support.

image

Copy link
Contributor

@liurenjie1024 liurenjie1024 left a comment

Choose a reason for hiding this comment

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

Thanks @kyteware for this pr! Let's hold for a while to resolve #1460 first

```toml
[dependencies]
iceberg-catalog-sql = "X.Y.Z"
sqlx = { version = "X.Y.Z", features = ["runtime-tokio", "sqlite"] }
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be confusing if sqlx 's version is same as iceberg-catalog-sql, I would prefer to change it to a different placeholder.


# Apache Iceberg SQL Catalog Official Native Rust Implementation

[![crates.io](https://img.shields.io/crates/v/iceberg.svg)](https://crates.io/crates/iceberg-catalog-sql)
Copy link
Contributor

Choose a reason for hiding this comment

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

In fact, iceberg-catalog-sql has not been published yet. See #1460

Co-authored-by: Kevin Liu <kevinjqliu@users.noreply.github.com>
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.

3 participants