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.

Sign up for GitHub

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

Jump to bottom

Unsupported fieldno index part #108

Closed
ArtDu opened this issue Jun 7, 2023 · 2 comments · Fixed by #109
Closed

Unsupported fieldno index part #108

ArtDu opened this issue Jun 7, 2023 · 2 comments · Fixed by #109
Assignees
@DifferentialOrange
Labels
bug Something isn't working teamE

Comments

@ArtDu
Copy link

ArtDu commented Jun 7, 2023

Now it crashes migrations with _tcf space

reproducer:

localhost:3301> ddl.check_schema(ddl.get_schema())
---
- true
...

localhost:3301> tmp = box.schema.create_space('tmp', {if_not_exists = true}); tmp:create_index('pri')
---
...

localhost:3301> ddl.check_schema(ddl.get_schema())
---
- null
- 'spaces["tmp"].indexes["pri"].parts[1].path: bad value (string expected, got number)'
...
@ArtDu ArtDu added the bug Something isn't working label Jun 7, 2023
@DifferentialOrange
Copy link
Member

Please, provide tarantool binary version

@ArtDu
Copy link
Author

ArtDu commented Jun 7, 2023

Please, provide tarantool binary version

As I see the problem version doesn't affect on problem

localhost:3301> _TARANTOOL
---
- 2.11.0-rc2-0-g535bac000-r549-gc64

@LeonidVas LeonidVas added the teamE label Jun 7, 2023
@DifferentialOrange DifferentialOrange self-assigned this Jun 7, 2023
DifferentialOrange added a commit that referenced this issue Jun 8, 2023
@DifferentialOrange
ddl: add basic fieldno path support for indexes
6c8a0c9 
It is the compatibility layer for Tarantool spaces created not through
ddl: users are not encouraged to create fieldno path indexes in ddl
schema explicitly, since we do not support them as sharding keys.

Closes #108
@DifferentialOrange DifferentialOrange mentioned this issue Jun 8, 2023
Merged
DifferentialOrange added a commit that referenced this issue Jun 8, 2023
@DifferentialOrange
ddl: add basic fieldno path support for indexes
6fb36d4 
It is the compatibility layer for Tarantool spaces created not through
ddl: users are not encouraged to create fieldno path indexes in ddl
schema explicitly, since we do not support them as sharding keys.

Closes #108
DifferentialOrange added a commit that referenced this issue Jun 8, 2023
@DifferentialOrange
Release 1.6.3
34fdabf 
Overview

  This release introduce  versioning support abd basic fieldno path
  support for indexes.

New features
  - Added versioning support (#105).

Buxfixes
  - Add basic fieldno path support for indexes (#108).
    It is the compatibility layer for Tarantool spaces created not
    through ddl: users are not encouraged to create fieldno path indexes
    in schema, since we do not support them as sharding keys.
AnaNek pushed a commit that referenced this issue Jun 8, 2023
@DifferentialOrange @AnaNek
ddl: add basic fieldno path support for indexes
f43005a 
It is the compatibility layer for Tarantool spaces created not through
ddl: users are not encouraged to create fieldno path indexes in ddl
schema explicitly, since we do not support them as sharding keys.

Closes #108
AnaNek pushed a commit that referenced this issue Jun 8, 2023
@DifferentialOrange @AnaNek
Release 1.6.3
fe55008 
Overview

  This release introduce  versioning support abd basic fieldno path
  support for indexes.

New features
  - Added versioning support (#105).

Buxfixes
  - Add basic fieldno path support for indexes (#108).
    It is the compatibility layer for Tarantool spaces created not
    through ddl: users are not encouraged to create fieldno path indexes
    in schema, since we do not support them as sharding keys.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DifferentialOrange DifferentialOrange

Labels
bug Something isn't working teamE
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Basic fieldno path support for indexes tarantool/ddl
3 participants
@DifferentialOrange @ArtDu @LeonidVas