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-8551] cache.put(already present key) on ms sqlserver fixed #5597

Closed
wants to merge 1 commit into from

Conversation

diuis
Copy link
Contributor

@diuis diuis commented Nov 21, 2017

The current (infinispan 9.1.3.Final) SQLServer implementation of AbstractTableManager (the class SQLServerTableManager) fails on updating cache entries.

@ghost
Copy link

ghost commented Nov 21, 2017

Can one of the admins verify this patch?

@gustavocoding
Copy link

ok to test

@diuis
Copy link
Contributor Author

diuis commented Nov 22, 2017

something went wrong with the execution test phase, @gustavonalle @ryanemerson could you take a look?

@ryanemerson
Copy link
Contributor

@diuis The tests aren't related, unfortunately our testsuite can be a bit flaky at times. I slightly changed your commit message to fit our style and integrated 0aaca8c

Thanks for your contribution 👍

@ryanemerson
Copy link
Contributor

@diuis FYI, I also backported the fix for the 9.1.x branch (060a4ef), so it will be available in 9.1.4.Final

@diuis
Copy link
Contributor Author

diuis commented Nov 22, 2017

@ryanemerson do you know when the 9.1.4.Final release will be ready?

@ryanemerson
Copy link
Contributor

@diuis Our aim is to release 9.2.0.Beta2 and 9.1.4.Final next week all being well

@diuis
Copy link
Contributor Author

diuis commented Nov 22, 2017

thanks @ryanemerson !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants