-
-
Notifications
You must be signed in to change notification settings - Fork 259
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
Unit tests showing some bugs (IMHO), i have discovered when using stateless updates #23
Conversation
…stateless update when using BeanPersistAdapter
Unit test wobu@5b732f1 for issue #25 |
Unit test wobu@fef37c7 for issue #26 |
Unit test wobu@1d8be8b for issue #27 |
…auses inserts instead of updates -> Primary key violation
Unit test wobu@e1f60a5 for issue #28 |
Commit wobu@336d964 fix issue #28 |
unintended closed pull request.. sry |
…ns invalidates cache correctly
added commits for issue #30 |
What is the state of these? Has Rob looked at them? |
Rob said he wants to fix the bugs till the next major release, cause he is doing a huge refactoring of the byte code enhancement. If he had already a look onto these bugs i don't know, but he fixed some other bugs in the last few days ;) |
Yes, I have been holding off looking at these stateless update tests/commits due to the enhancement changes which changed the stateless update implementation details almost completely (as there is no longer a need to specify the properties to update etc as the enhancement takes care of that). So yes, if there is anything in here that is urgent lets raise that but if we can hold off to 'R4' which is perhaps a few weeks away that would be good. |
this pull requests contains a commit for a caching bug. Generally the caching bugs i have found are for me a greater pain in the ass :( Is it possible that you will at least have a look on them? |
There is already a fix for issue 30 in master - I have just updated that related commit linking to the commit in master. Was there another fix? I didn't see one when I browsed through this just now. |
i meant the issues
but i couldn't provide fixes for them :( |
I have merged all of these test cases into the develop-v4 branch. All tests passing - closing this now. |
… to flush last ChangeLog changes at PreCommit
No description provided.