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

[backport 3.1] config: use vshard-ee if available #10144

Conversation

Totktonada
Copy link
Member

(This is a backport of PR #10129 to release/3.1, future 3.1.1 release.)


The new closed-source vshard-ee module was recently introduced. It is based on the open-source vshard module and, as far as I know, provides the same API.

Let's configure vshard-ee in the same way as vshard if sharding is enabled in tarantool's configuration.

Prefer vshard-ee if both are available.

Fixes https://github.com/tarantool/tarantool-ee/issues/815

Usage example:

```lua
local loaders = require('internal.loaders')
local vshard = loaders.require_first('vshard-ee', 'vshard')
```

The function is for internal use.

It would be nice to have something of this kind in the public API, but
I'm not going to solve it within this patch.

Needed for tarantool/tarantool-ee#815

NO_DOC=no public API changes
NO_CHANGELOG=see NO_DOC

(cherry picked from commit 3abd4f9)
The new closed-source `vshard-ee` module was recently introduced. It is
based on the open-source `vshard` module and, as far as I know, provides
the same API.

Let's configure `vshard-ee` in the same way as `vshard` if sharding is
enabled in tarantool's configuration.

Prefer `vshard-ee` if both are available.

Fixes tarantool/tarantool-ee#815

@TarantoolBot document
Title: config: vshard-ee is now supported

The declarative configuration supports `vshard-ee` in addition to
`vshard` since Tarantool 3.1.1 and 3.2+.

`vshard` is mentioned in the documentation at least [here][1]. All such
places should be updated to mention both `vshard-ee` and `vshard`.

[1]: https://www.tarantool.io/en/doc/latest/reference/configuration/configuration_reference/#sharding

(cherry picked from commit 29519c7)
@Totktonada Totktonada requested a review from a team as a code owner June 18, 2024 13:10
@Totktonada Totktonada removed the request for review from a team June 18, 2024 13:10
@Totktonada
Copy link
Member Author

The reviews are done in PR #10129. This is a backport of the same patches without any changes. I'll proceed with them.

@Totktonada Totktonada added the full-ci Enables all tests for a pull request label Jun 18, 2024
@coveralls
Copy link

Coverage Status

coverage: 87.076% (+0.01%) from 87.063%
when pulling 89fa2a7 on Totktonada:backport-3.1/ghe-815-config-support-vshard-ee
into ae3231a
on tarantool:release/3.1
.

@Totktonada Totktonada merged commit a267c3f into tarantool:release/3.1 Jun 18, 2024
91 checks passed
@Totktonada Totktonada deleted the backport-3.1/ghe-815-config-support-vshard-ee branch June 18, 2024 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full-ci Enables all tests for a pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants