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

crud: fix options for SelectRequest #323

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

oleg-jukovec
Copy link
Collaborator

The patch fixes a typo that made it impossible to setup SelectOpts.After, SelectOpts.BatchSize and SelectOpts.ForceMapCall.

What has been done? Why? What problem is being solved?

I didn't forget about (remove if it is not applicable):

Related issues:

Part of #320

@siller174
Copy link

when could you merge it?

@oleg-jukovec
Copy link
Collaborator Author

when could you merge it?

After 2 reviews. It could take a while.

Copy link

@DerekBum DerekBum left a comment

Choose a reason for hiding this comment

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

Lgtm. Just needs a small changelog fix

CHANGELOG.md Outdated Show resolved Hide resolved
The patch fixes a typo that made it impossible to setup
SelectOpts.After, SelectOpts.BatchSize and SelectOpts.ForceMapCall.

Part of #320
@oleg-jukovec oleg-jukovec merged commit f56fb90 into master Aug 3, 2023
22 checks passed
@oleg-jukovec oleg-jukovec deleted the oleg-jukovec/gh-320-fix-after branch August 3, 2023 09:13
@siller174
Copy link

Thank you. Could you create new release?

@oleg-jukovec
Copy link
Collaborator Author

At the moment we are not ready for the next major release:

https://github.com/tarantool/roadmap-internal/issues/187

@siller174
Copy link

siller174 commented Aug 3, 2023

I don't have access to this repository. Could you make m.b. minor release? Or Do you know when will be major release?

Because using

github.com/tarantool/go-tarantool/v2 v2.0.0-20230803091303-f56fb906e20a

it's strange :)

@oleg-jukovec
Copy link
Collaborator Author

We'll make a patch release: #325

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.

4 participants