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

ISPN-8719 KeySet.(iterator|spliterator|stream) not compatible with #5896

Merged
merged 1 commit into from Apr 7, 2018

Conversation

wburns
Copy link
Member

@wburns wburns commented Apr 4, 2018

versions before 9.2

  • Added iterator methods to Codecs
  • Put appropriate methods on each Codec version

https://issues.jboss.org/browse/ISPN-8719

@wburns wburns added the Preview label Apr 4, 2018
@wburns wburns force-pushed the ISPN-8719_key_converter branch 3 times, most recently from 3133fca to eab7315 Compare April 4, 2018 21:01
@wburns
Copy link
Member Author

wburns commented Apr 5, 2018

I have added tests and they all seem to work except for ProtocolVersion.2.3 where for some reason the result is getting jumbled and the messageId is found to be 0 causing a NPE. I am not sure where the bug is as the server side seems to be sending the correct bytes back. I need to look into it further still.

@wburns
Copy link
Member Author

wburns commented Apr 6, 2018

So the cause with version 2.3 is due to the fact that the retrieveEntries in newer clients always uses the 2.5 protocol even when 2.3 is set. Also newer clients have never been able to communicate to any server of version 2.2 or older in general. So that means no client since 8.1 could ever talk to any version older than that.

@wburns wburns force-pushed the ISPN-8719_key_converter branch 4 times, most recently from 0844d07 to ad864bb Compare April 6, 2018 04:41
@wburns
Copy link
Member Author

wburns commented Apr 6, 2018

Also fixed a 2.5+ client talking to 2.4 since it was always passing meta but that is not returned in 2.4. This would make reading entries return incorrect values and/or possibly hang.

@wburns wburns force-pushed the ISPN-8719_key_converter branch 2 times, most recently from c0a0115 to 495d8d4 Compare April 6, 2018 05:02
@wburns wburns added Urgent! and removed Preview labels Apr 6, 2018
@wburns wburns force-pushed the ISPN-8719_key_converter branch 3 times, most recently from e93b99c to ea7bba1 Compare April 6, 2018 15:24
versions before 9.1

* Added iterator methods to Codecs
* Put appropriate methods on each Codec version
@wburns wburns changed the title ISPN-8719 ToEmptyBytesKeyValueFilterConverter not compatible with ISPN-8719 KeySet.(iterator|spliterator|stream) not compatible with Apr 6, 2018
@tristantarrant
Copy link
Member

This looks great. We will eventually need some integration tests for testing against combinations of servers/clients.

@tristantarrant tristantarrant merged commit b4bb667 into infinispan:master Apr 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants