Skip to content

Conversation

LeonidVas
Copy link
Contributor

@LeonidVas LeonidVas commented Oct 22, 2020

  • For all queue drivers, the "new" method now checks for the presence of indexes created by the "create_space" method.
  • If a space is creating with the "if_not_exist = true" option and a space with such name already exists (and this is not a tube of the required type), a conflict will be detected and an error will be thrown. This is necessary because we don't want to create additional indexes for the existing space and repurpose it into the "tube".

Closes #62

@ChangeLog Added space schema validation to all core drivers when creating a tube object. If a space is creating with the "if_not_exist = true" option and a space with such name already exists (and this is not a tube of the required type), a conflict will be detected and an error will be thrown.

@LeonidVas LeonidVas force-pushed the picodata/validate-space branch 2 times, most recently from 2d9a9d3 to ec72049 Compare October 22, 2020 22:15
For all queue drivers, the "new" method now checks for the presence
of indexes created by the "create_space" method.

Co-authored-by: Leonid Vasiliev<lvasiliev@tarantool.org>

Part of #62
@LeonidVas LeonidVas force-pushed the picodata/validate-space branch 2 times, most recently from 594a48b to 31135cb Compare October 23, 2020 08:31
@LeonidVas LeonidVas requested a review from Totktonada October 23, 2020 08:49
@LeonidVas LeonidVas self-assigned this Oct 23, 2020
Copy link
Contributor

@Totktonada Totktonada left a comment

Choose a reason for hiding this comment

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

I like the idea to offer extra safely over mistakes around space / tube names. I don't see any problems in the implementation. LGTM.

If a space is creating with the "if_not_exist = true" option and a
space with such name already exists (and this is not a tube of
the required type), a conflict will be detected and an error will
be thrown.
This is necessary because we don't want to create additional indexes
for the existing space and repurpose it into the "tube".

Closes #62
@LeonidVas LeonidVas force-pushed the picodata/validate-space branch from 31135cb to e21bb15 Compare October 27, 2020 18:02
@LeonidVas LeonidVas merged commit d709208 into master Oct 27, 2020
@LeonidVas LeonidVas deleted the picodata/validate-space branch October 27, 2020 18:55
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.

tube:create() should check that a space with this name exists

2 participants