-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Release version 0.8.0 #678
Comments
RC1 is available: https://github.com/brianfrankcooper/YCSB/releases/tag/0.8.0-RC1 Please chime in if you can help test and which platforms you will work on. THANKS! |
I should have Accumulo and works-on-windows tested this week. |
Thanks @busbey! Would you be available to also check the HBase10 binding? For this release candidate 0.8.0-RC1: |
Ran ycsb 0.8.0-RC1 against workloada for both HASH key and HASH_AND_RANGE key on dynamodb-binding. Looks good. |
@cmatser Yes, I'll check it tomorrow. Thanks for pinging me. |
@cmaster - I will try and big up my notes in Tarantool. |
I am having a hard time building due to the
Anyone else have this issue? I tried both the OpenJDK 8 and Oracle JDK 8 with the same error. |
It's supposed to just start without native sigar if needed. What OS and version? |
Fedora 23. There are a number of Cassandra tickets that if the machine is older the timeout might need to be increased. This desktop is getting older but not that old. I noticed that this test is not running (that I could find) as part of the Travis builds. Any idea how that was turned off? |
Building with Java 7 works. |
Java 7 will totally skip testing the cassandra2 module, so it should work fine. :) |
I ran the basic set of workloads against Tarantool and saw no issues. |
@yuyantingzero Thank you. Can you verify what version of DynamoDB? @allanbank Thank you. Can you verify Tarantool version used? |
Tarantool 1.6.6-249 |
@cmatser I checked the S3 binding and it works well. |
@zx-zheng Thank you. Is there are specific version of S3 you tested with? |
@cmatser No, I tested with our private S3 compatible system. |
@cmatser aws sdk version is 1.10.48. Dynamodb is managed and doesn't have a version. |
For fun I tested the mongodb and mongodb-async bindings again the following versions of MongoDB:
No issues. |
I'll check latest Tarantool version |
We're nearing the end of the week and I wanted to get some input on the remaining pieces that still need to be tested. I didn't get strong commitments on the following bindings. Please reply to this thread if you can pickup one or more of these bindings to test for the release: These bindings are tested and ready for the release: These bindings are in process by the following people. Please reply to this thread with any updates. Thanks for all the help. Keeping YCSB up-to-date would not happen without the contributions from all of you. |
@cmatser Yes, I do. During testing I noticed two issues that deserve attention:
Given that, I would like to see #684 to address those two issues and clean up some other things as well. |
I'm testing @daschl could you test the current couchbase binding included in this release candidate? |
@cmatser will check and report back by EOD. |
hbase10 binding checks out fine. tested against a cluster running a derivative of HBase 1.0.0 (CDH5.5.2, specifically) |
It works with Tarantool 1.6.8 and Tarantool 1.7 (Alpha) |
@busbey couchbase1 looks good (tested against Server 4.1)! Do you think we can get in couchbase2 for 0.8 as well? |
There's already a lot going into this release, I would just aim for the start-of-May release so that we needn't rush getting things done. Usually the creation of the staging branch marks feature-complete for the release. Adding support for a new major version of a datastore feels like a new feature to me. |
RC3 is available: https://github.com/brianfrankcooper/YCSB/releases/tag/0.8.0-RC3 This release candidate contains the recent Elasticsearch fix for race condition. @jasontedor, can you please test this release candidate on the Elasticsearch platform and verify that I've pulled in your commit appropriately. Thanks! Good catch on the race condition! They can be difficult to diagnose and frustrating for the end user. |
RC2 works as expected from binary form when following our Windows instructions. When building from source I had to skip tests with JDK8 due to the same timeouts in Cassandra Unit as @allanbank ran into. Those tests built successfully for the 0.7.0 RC, so I'm going to try to get a patch together to tune the timeouts. I don't think this issue needs to hold up the release, we'll just have to be mindful if folks with older windows machines start showing up asking for help. |
I tested RC3 with cassandra 2.2.5. Looks good. First draft of release notes: https://gist.github.com/cmatser/1fc8a4aa451947dc8df73c1302fc45e4 I think we'll be ready to go with the release after I hear back on Accumulo and Elasticsearch and their versions. |
Elasticsearch is good. I tested RC3 again against a local node, a remote node, and a remote three-node cluster and all is well. |
Thanks @jasontedor! What version? |
we should include #692 once it merges. |
👍 accumulo binding works as expected. I ran into a couple of issues, but not things specific to this release or that we have a solid approach to fixing.
|
We should still wait on #692 before releasing. The release notes look good, a couple of feedback points:
|
The list of Cassandra related improvements should call out #614, since it added compatibility for Cassandra 3. It was pulled in via the merge of #669 (which is already listed), but we should make that more obvious. It would have been great to get a test against a Cassandra 3 cluster in, since that should work now and is new, but I don't think I have time before #692 lands. Anyone else have ~30 minutes to run through things? |
@busbey thanks for going over the release notes. I'm a little unsure how to word the Cassandra 3 update. How do we say we have updates to support Cassandra 3, but it is untested and unverified? There's still issue #677 to verify Cassandra 3. What version of Accumulo did you test? ...waiting on the #692 merge. Then I'll create RC4 and ask you to retest HBase10. |
We could just list Cassandra 3.x via the cassandra2-cql binding under untested (presuming I have the name right) |
I tested with Accumulo 1.6.0. If folks end up here and would like precise details, it was the vendor packaged 1.6.0-cdh5.1.4 running on top of CDH 5.7.0. |
@cmatser Geode was tested on 1.0.0-incubating.M1 |
RC4 is available: https://github.com/brianfrankcooper/YCSB/releases/tag/0.8.0-RC4 This release candidate contains the recent HBase10 fix for synchronization. @busbey , can you please test this release candidate on the HBase platform and verify that I've pulled in the appropriate commit appropriately. Thanks! Thanks @sbawaska, I've captured the Geode version. I believe this HBase test is the last hurdle before closing out the release. @busbey, should we call out attention to the follow-on work for HBase regarding the syncronization fix? |
testing RC4 now. I don't think we need to call out the follow-on work. It doesn't impact correctness and any impact it has on performance will be seen in the next release. (we expect negligible impact) |
+1 tested RC4 against |
release notes look good too. 👍 |
YCSB 0.8.0 is out https://github.com/brianfrankcooper/YCSB/releases/tag/0.8.0 Will make the official announcements, wiki, and readme updates soon. |
This is the planned announcement: https://gist.github.com/cmatser/ff88389c26f2cd0b9eddc4734aedc007 |
I personally don't think the style enhancements need to be called out at all, but I can see why some folks might disagree. Please remember to call the Apache projects "Apache Foo" in their first occurrence in the message.
I do the windows build testing every release, I don't think it's worth calling out. I also only use the basic binding when testing on windows.
Since Cassandra3 didn't make it to supported, I'd recommend phrasing this more along the lines of "Cassandra binding updated to DataStax driver 3.0.0, tested with Cassandra 2.2.5." In the announcement specific to the Cassandra community, I'd recommend including a note about how we'd love some help testing it against Cassandra 3 for the next release. |
@cmatser the updates to the announcement look good to me. |
The change was to skip the Accumulo tests on Windows entirely because we can't initialize the minicluster. Can probably leave it out. |
other than that Accumulo bit, 👍 |
Announced on the following lists: Aerospike Forum Done. Thanks to everyone for helping out. |
For the next release, 0.8.0, the following items have already been merged:
Breaking changes:
Bindings to test:
Release plan:
The text was updated successfully, but these errors were encountered: