Skip to content

Conversation

@babyTsakhes
Copy link
Collaborator

  • The flags of functions IPROTO_FEATURE_IS_SYNC and IPROTO_FEATURE_INSERT_ARROW protocol function flags were added to greeting negotiation.
  • Fixed tests that test for flags by adding a byte sequence.

Added #466

Copy link
Collaborator

@oleg-jukovec oleg-jukovec left a comment

Choose a reason for hiding this comment

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

Please, squash the commits:

iproto: add missing IPROTO feature flags to greeting negotiation

The flags of functions IPROTO_FEATURE_IS_SYNC and IPROTO_FEATURE_INSERT_ARROW protocol function flags were added in the welcome, fixed tests that test for flags by adding a byte sequence.

Fixed https://github.com/tarantool/go-tarantool/issues/466

iproto: add missing IPROTO feature flags to greeting negotiation

The flags of functions IPROTO_FEATURE_IS_SYNC and IPROTO_FEATURE_INSERT_ARROW protocol function flags were added in the welcome, fixed tests that test for flags by adding a byte sequence.

Added https://github.com/tarantool/go-tarantool/issues/466

->

iproto: add missing IPROTO greeting flags

The greeting flags of functions IPROTO_FEATURE_IS_SYNC and
IPROTO_FEATURE_INSERT_ARROW protocol function flags were added.

Closes #466

Into a single-well written commit according to the rules:

https://www.tarantool.io/en/doc/latest/contributing/developer_guidelines/#how-to-write-a-commit-message

@babyTsakhes babyTsakhes force-pushed the babyTsakhes/gh-466-add-missing-IPROTO-feature-flags branch from 176ad4e to fde83ae Compare October 30, 2025 18:03
The greeting flags of functions IPROTO_FEATURE_IS_SYNC and
IPROTO_FEATURE_INSERT_ARROW protocol function flags were added.

Closes #466
@babyTsakhes babyTsakhes force-pushed the babyTsakhes/gh-466-add-missing-IPROTO-feature-flags branch from fde83ae to 927276c Compare October 30, 2025 18:05
Copy link
Collaborator

@oleg-jukovec oleg-jukovec left a comment

Choose a reason for hiding this comment

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

Thank you for the patch!

@oleg-jukovec oleg-jukovec merged commit 3e75868 into master Oct 31, 2025
42 of 43 checks passed
@oleg-jukovec oleg-jukovec deleted the babyTsakhes/gh-466-add-missing-IPROTO-feature-flags branch October 31, 2025 13:30
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