Skip to content
This repository has been archived by the owner on Mar 31, 2023. It is now read-only.

alberto/release-nordix/1.13.0/feature/GEODE-10300 #2

Open
wants to merge 2 commits into
base: release-nordix/1.13.0
Choose a base branch
from

Conversation

albertogpz
Copy link
Collaborator

Remove temporary solution for ADPPRG-97343 (C++ native client: Allow locator responses greater than 3000 bytes) by back-porting solution for GEODE-10300.

…ndle big number of kvdb servers"

This reverts commit 6aa104b.
apache#970)

* GEODE-10300: Fix locator response size limit in C++ client

If a response message from the locator to the C++ native client
is longer than 3000 bytes the C++ native client will only
read the first 3000 bytes.

* GEODE-10300: Updated after review

* GEODE-10300: Updated after review

* GEODE-10300: Updated after another review

* GEODE-10300: Updated after some more reviews

* GEODE-10300: Some more changes after review.

* GEODE-10300: Small change after review

* GEODE-10300: Remove unneeded space
Copy link
Collaborator

@gaussianrecurrence gaussianrecurrence left a comment

Choose a reason for hiding this comment

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

  • Code changes looks OK
  • Tests are passing

@metal3-io-bot
Copy link
Member

@albertogpz: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants