-
Notifications
You must be signed in to change notification settings - Fork 200
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
Added tests for bulk ops atomicity #1294
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall the test look good. If the overall test takes quite long to run, can you mark these as ignore for now and then later tag them so that these can be run conditionally (like mentioned here: http://www.scalatest.org/user_guide/using_the_runner#filtering). Also update the spark/store links to the latest merged revisions.
Can be run conditionally and some can be converted to hydra
Can one of the admins verify this patch? |
8b43301
to
2b254d9
Compare
2c254f0
to
0f2888f
Compare
a466d26
to
ea127bd
Compare
99ec79c
to
c7b84fa
Compare
Changes proposed in this pull request
Tests for insert/update/delete bulk ops
Considered row buffer, reduced oldEntryInterval and eviction scenarios
Patch testing
(Fill in the details about how this patch was tested)
ReleaseNotes.txt changes
(Does this change require an entry in ReleaseNotes.txt? If yes, has it been added to it?)
Other PRs
(Does this change require changes in other projects- store, spark, spark-jobserver, aqp? Add the links of PR of the other subprojects that are related to this change)