search

tarantool / migrations

BSD 2-Clause "Simplified" License
21 stars 5 forks source link

api: allow to set storage timeout #67

Closed DifferentialOrange closed 1 year ago

DifferentialOrange commented 1 year ago

This patchset introduces various test fixes, as well as ability to set storage timeout.

Before this patch, storage net.box call for migration evaluation was strictly limited to 3600 seconds with a comment "timeout should be disabled ... [since] migrations might take long time". But there are users who prefer migrations slower than one hour to reduce the load on production cluster, and such approach is impossible with current hardcoded timeout implementation. After this patch, one will be able to set up call timeout with clusterwide configuration section.

migrations:
  options:
    storage_timeout: 43200 # in seconds

Since default is 3600 seconds either way, this change should not affect existing users.

Closes #63 Closes #66

DifferentialOrange commented 1 year ago

add changelog entries for test fixes

Added to changelog and release notes. Also clarified the reason behind the first fix: it's actually HTTP client (core Tarantool)-related thing, not a tarantool/http one.