Skip to content

Conversation

jseldess
Copy link
Contributor

Fixes #1589

@jseldess jseldess requested a review from m-schneider August 24, 2017 02:13
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@jseldess
Copy link
Contributor Author

@m-schneider, finally adding this as a known limitation for 1.0. Also adding it as a limitation ion 1.1, but once cockroachdb/cockroach#17860 lands, I can remove that.

@cockroach-teamcity
Copy link
Member

@cockroach-teamcity
Copy link
Member

@m-schneider
Copy link

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, all commit checks successful.


v1.0/known-limitations.md, line 286 at r1 (raw file):

## Dropping an interleaved index prevents DDL operations

When you [drop an interleaved index](drop-index.html) on a table, future DDL operations on the table will fail, for example:

The limitation is only when you drop an interleaved index on the same table. In your example: INTERLEAVE IN PARENT t1 (id2); would fail, but if the parent was t2 or any other table we wouldn't have the problem.


Comments from Reviewable

@jseldess
Copy link
Contributor Author

v1.0/known-limitations.md, line 286 at r1 (raw file):

Previously, m-schneider (Masha Schneider) wrote…

The limitation is only when you drop an interleaved index on the same table. In your example: INTERLEAVE IN PARENT t1 (id2); would fail, but if the parent was t2 or any other table we wouldn't have the problem.

Thanks, Masha. Take another look.


Comments from Reviewable

@cockroach-teamcity
Copy link
Member

@m-schneider
Copy link

:lgtm:


Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks pending.


v1.0/known-limitations.md, line 286 at r1 (raw file):

Previously, jseldess wrote…

Thanks, Masha. Take another look.

Looks good!


Comments from Reviewable

@cockroach-teamcity
Copy link
Member

@jseldess jseldess merged commit ceee5bc into master Aug 24, 2017
@jseldess jseldess deleted the known-limitation branch August 24, 2017 16:28
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