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

docs: update SQL in the design document #9651

Merged
merged 1 commit into from
Sep 30, 2016
Merged

Conversation

knz
Copy link
Contributor

@knz knz commented Sep 30, 2016

This change is Reviewable

A preliminary design can be found in the [Go source documentation](https://godoc.org/github.com/cockroachdb/cockroach/sql).
# SQL

Each node in a cluster can accept SQL client connections. It supports
Copy link
Collaborator

Choose a reason for hiding this comment

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

s/It/CockroachDB/g

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done.

and SERIALIZABLE. The other traditional SQL isolation levels are
not supported and there is no plan to implement them in the future.

- CockroachDB implements its own SQL type system (cf. `typing.md`)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you provide a full link to typing.md?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done.

- CockroachDB exclusively implements MVCC-based consistency for
transactions, and thus only supports SQL's isolation levels SNAPSHOT
and SERIALIZABLE. The other traditional SQL isolation levels are
not supported and there is no plan to implement them in the future.
Copy link
Member

Choose a reason for hiding this comment

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

They are supported but internally always map to either SNAPSHOT or SERIALIZABLE.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

and SERIALIZABLE. The other traditional SQL isolation levels are
not supported and there is no plan to implement them in the future.

- CockroachDB implements its own SQL type system (cf. `typing.md`)
Copy link
Member

Choose a reason for hiding this comment

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

Make this a link.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@knz knz merged commit 04aa68a into cockroachdb:master Sep 30, 2016
@knz knz deleted the rewrite-md branch September 30, 2016 19:44
@jseldess jseldess mentioned this pull request Oct 5, 2016
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