Skip to content

Add support of numerical index path#397

Merged
dkasimovskiy merged 3 commits intomasterfrom
gh-395-1
Jun 8, 2023
Merged

Add support of numerical index path#397
dkasimovskiy merged 3 commits intomasterfrom
gh-395-1

Conversation

@iDneprov
Copy link
Copy Markdown
Contributor

@iDneprov iDneprov commented Jun 7, 2023

Change TarantoolIndexPartMetadata.path type from String to Object to store String or int values

I haven't forgotten about:

  • Tests
  • Changelog
  • Documentation
  • Commit messages comply with the guideline
  • Cleanup the code for review. See checklist

Related issues:

Closes #395

Change TarantoolIndexPartMetadata.path type from String to Object to store String or int values

Closes #395
@iDneprov iDneprov changed the title WIP Add support of numerical index path Add support of numerical index path Jun 7, 2023
@iDneprov iDneprov requested review from ArtDu and bitgorbovsky June 7, 2023 12:07
Copy link
Copy Markdown
Collaborator

@akudiyar akudiyar left a comment

Choose a reason for hiding this comment

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

LGTM, only a small changelog fix

Co-authored-by: Alexey Kuzin <akudiyar@gmail.com>
@iDneprov iDneprov requested review from akudiyar and dkasimovskiy June 8, 2023 07:33
@dkasimovskiy dkasimovskiy merged commit 90e3bc4 into master Jun 8, 2023
@dkasimovskiy dkasimovskiy deleted the gh-395-1 branch June 8, 2023 07:47
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.

Parse index path value in ddl response not only as String

3 participants