-
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 0.14.0 #1117
Comments
Just curious, what testing work is needed? Is this a matter of manually testing the bindings, the individual features/PRs, or both? |
yep! we'll need to test a number of bindings and any specific fixes that are supposed to be present. The first step is to get a list up of which bindings / features /etc. It's on my todo list, but I've been a bit swamped on a different project lately. |
generally speaking, testing a binding should be straight forward. It's essentially:
using either the binding specific tarball or the omnibus is fine, but which one got used should probably be noted in the results. |
merged #908 |
would like to get #1148 "Update HBase 2 version to 2.0.0 GA" in for polish on the hbase addition |
Another modified store I just merged:
|
found some headers missing before making the staging branch. put up #1150 to fix it. |
excellent catch! |
Release candidate 1 is up: https://github.com/brianfrankcooper/YCSB/releases/tag/0.14.0-RC1 |
@busbey Just tested 0.14.0-RC1 with Apache Cassandra and @YugaByte datastores with no issue. Here are the test config and results: Datastore setup:
YCBS client setup:
YCSB workload parameters:
Apache Cassandra config:
Results: YugaByte DB config:
Results: Please let me know if you need more details or the full output log. |
Thanks @robertpang! that'll do nicely. |
I sent pings for testing to the user@ lists for:
|
known issues for this release should include #1155 |
@upthewaterspout and I tested the Geode datastore versions 1.2.0, 1.3.0, and 1.6.0 (latest) against the Geode driver. Everything looks good! |
thanks geode folks! |
I tested 0.14.0-RC1 with Kudu (single node) on CentOS 6.6 and tweaked with some workload parameters as below: Nothing looks wrong! |
thanks @fwang29! what version of Kudu did you use? |
@busbey NP! Sorry, I forgot to mention it. It was version 1.8.0. |
I tested the mongodb binding and had no issues. YCSB build:
Host details:
Client details:
Database setup:
YCSB workload parameters:
Disclaimer: this was and a "fire and forget" run with no tuning Results for mongodb client
Results for mongodb-async client
|
I'd like to wrap up this RC by the end of the week. Thanks so much to the folks who have tested things so far! IMHO we already have enough tested datastore bindings to close things out. Since I have most of the testing for the Apache HBase related changes done, I think I'll make a go of finishing that up before closing things up. I can test out that basics work on windows as well, unless someone else wants to take it on. Bindings currently slated to be listed as "untested / your milage may vary", with a ping to folks who showed up on related PRs:
|
I have this continuously running in our test environment and see no problem. |
@haih-g could you have it test specifically against the 0.14.0-RC1 tag? |
@busbey where I can get the actual list of untested bindings? |
My comment from ~3 days ago is still fairly up to date. IIRC, only Google Datastore has since been tested. I'm probably only going to test the HBase 1.2+ bindings, which would mean HBase 0.98 and HBase 1.0 will also go into the list. Is that what you're looking for @isuntsov-gridgain? |
Although I was tagged, I won't be able to test the Accumulo bindings without detailed step-by-step instructions... I'm not a YCSB user and know next to nothing about YCSB. However, with detailed step-by-step testing instructions, I'd happily launch an Accumulo instance to test against. |
@ctubbsii see the README, e.g https://github.com/brianfrankcooper/YCSB/tree/master/accumulo1.8. It's very straightforward. Just "load" and then "test" for a workload. Testing one is likely "good enough" (you don't need to do workloads A through F). |
@joshelser Thanks. I didn't realize there was a README in the subdirectory with detailed instructions. Will use those. 😺 |
Hi Sean,
I have tested my s3 changes with 0.14.0 RC1.
My testing have not revealed any problems.
Best regards,
Boris Osher
(408) 368-9089
…On Mon, Jun 4, 2018 at 10:26 AM, Sean Busbey ***@***.***> wrote:
I'd like to wrap up this RC by the end of the week. Thanks so much to the
folks who have tested things so far!
IMHO we already have enough tested datastore bindings to close things out.
Since I have most of the testing for the Apache HBase related changes done,
I think I'll make a go of finishing that up before closing things up.
I can test out that basics work on windows as well, unless someone else
wants to take it on.
Bindings currently slated to be listed as "untested / your milage may
vary", with a ping to folks who showed up on related PRs:
- Accumulo 1.6, 1.7, 1.8 ***@***.*** <https://github.com/joshelser>
@phrocker <https://github.com/phrocker> @hinchliff
<https://github.com/hinchliff> @ctubbsii <https://github.com/ctubbsii>
have time to take a look?)
- Aerospike ***@***.*** <https://github.com/tiboratAS> have time to
take a look?)
- Azure DocumentDB ***@***.*** <https://github.com/k1xme> have time to
take a look?)
- ArangoDB3 ( @mpv1989 <https://github.com/mpv1989> @diogojfernandes
<https://github.com/diogojfernandes> have time to take a look?)
- Elastic Search 5 ( @risdenk <https://github.com/risdenk> @jasontedor
<https://github.com/jasontedor> @levaphenyl
<https://github.com/levaphenyl> have time to take a look?)
- Google Bigtable ***@***.*** <https://github.com/sduskis>
@igorbernstein2 <https://github.com/igorbernstein2> @mbrukman
<https://github.com/mbrukman> have time to take a look?)
- Google Cloud Spanner ***@***.*** <https://github.com/siamaktz>
@samizuh <https://github.com/samizuh> @hostirosti
<https://github.com/hostirosti> have time to take a look?)
- Google Datastore ***@***.*** <https://github.com/haih-g> have time to
take a look?)
- JDBC ***@***.*** <https://github.com/k1xme> @risdenk
<https://github.com/risdenk> have time to take a look?)
- MaprDB, ***@***.*** <https://github.com/rohanjayaraj>
have time to take a look?)
- Memcached ***@***.*** <https://github.com/solganik> have time to take
a look?)
- Redis ***@***.*** <https://github.com/j3ns> have time to take a look?)
- s3 ***@***.*** <https://github.com/bosher> @risdenk
<https://github.com/risdenk> have time to take a look?)
- Solr, Solr6 ***@***.*** <https://github.com/risdenk> have time to take
a look?)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1117 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABXzlYhNxXlK2OXQSM0uy9QHa-_Kgn4rks5t5W3DgaJpZM4S-Bxn>
.
|
Issues I found testing Accumulo 1.9.1:
I'm not sure what to look for in the testing output, but everything seemed to work fine if I set accumulo.1.8.version to 1.9.1 and ran I tested using https://github.com/apache/fluo-uno using workload A on a single machine, with Apache Accumulo 1.9.1, Apache Hadoop 2.7.6, and Apache ZooKeeper 3.4.12 |
are the Accumulo version numbers needed for the release or would they be fine to update for next time? |
I think the main risk with releasing without the updated Accumulo version numbers is possible confusion. I would rate the risk low. |
@busbey actually I'm not familiar with any distributed systems from this list so it will be the random choice. |
Guys, I see in YCSB/pom.xml (0.14.0-staging): It is a version that was released in 2014. With the 4.1+ I have compilation problems. Is it OK? |
If it works with the version advertised in YCSB then it's okay for the release. if things are tested then we expressly call out what version(s) were tested for the release notes. ("advertised" here means either a) mentioned in the README, b) mentioned in prior release notes, c) version in the pom For this sort of thing usually I'd suggest making sure there's a ticket about needing to update. Or if you want to and have time to fix it now just start with a PR. We can include an update in the next release. I'd look for feedback from whoever maintains Aerospike about wether it's worth keeping compatibility with both v3 and v4. |
@busbey Tested 0.14.0-RC1 with MapR-DB and MapR-JSONDB datastores and found no issues. Below are the configs and results : Datastore setup
YCBS client setup
YCSB workload parameters
MapR Database Setup
Results for maprdb client
Results for maprjsondb client
|
I've finished with Aerospike.
YCBS client setup
Aerospike server version: 4.2.0.3 Results for Aerospike client: |
Redis - done!
YCBS client setup
load run |
in evaluating the RC I found that the big tarball doesn't have a README.md included, filed #1171. It's been broken for ~4 years so I don't think it needs to be fixed for this release. someone shout if they'd prefer otherwise. |
the hbase098 binding instructions refer to a binding that doesn't exist instead of hbase098. filed #1172. I don't think this should be a blocker. |
tested against HBase 1.2 (CDH 5.14.2) (via found #1173 [hbase20] no slf4j implementation; hbase client doesn't log. it doesn't seem like a blocker, since none of the information we miss from logging is critical and it can be worked around by adding an slf4j logging-binding to the classpath. |
tested against HBase 2.0 (CDH 6.0.0-beta1) (via #1173 applies there too. |
@busbey: release notes is great! I have 2 minor comments:
|
Thanks @petersomogyi! I've corrected both of those now. |
@busbey thanks a lot for the release note! |
good catch @jojochuang! fixed now. thanks. |
Unless anyone else has something they'd like to see changed, I plan to push the release in about an hour, FYI. |
PR up to point folks to the new release from our landing page: #1174 . if someone could take a look please. |
Closing this release out. I don't have the time ATM to do release announcements across the various mailing lists. If anyone else wants to pick that up let me know and I'll point you at the template. |
follow up from #981. There's already a fair bit of churn on master in the ~9 months since the 0.13.0-staging branch started.
Additionally, we have a good deal of testing work left over that never happened in the 0.13.0 RC process.
The text was updated successfully, but these errors were encountered: