-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Why not use ScyllaDB? #197
Comments
we're not against it, but we have been using Cassandra for quite some time now. If you'd like to provide a proof of concept or add a scyllaDB implementation, we're all for it. |
Hi guys, we are launching a scylladb backed cluster right now, we just have a few issues to solved :
Otherwise it should be fully compatible. |
@jbdalido right before we open sourced, we were using DateTieredCompactionStrategy, there is some performance benefit but I'd be lying if we said we exhaustively checked performance benchmarks. You should be fine. We would love to see PRs for any ScyllaDB-specific modifications. |
@badiib Thanks, we will do it soon, it's in production right now, the only things we are missing is the Custom Index https://github.com/uber/jaeger/blob/master/plugin/storage/cassandra/cassandra3v001-schema.sh#L190, unavailable in ScyllaDB. if it's ok without it, i'll PR asap the removal of the proto-version and the schema specific file. |
Has there been a recent update on this I see that the PR has stalled? |
@yurishkuro this might be faster than Cassandra. Can't vouch for compatibility, but seems worth looking into. |
Any news ? |
Unless @jbdalido updates this, no. |
any updates? @jbdalido |
I found the new scylladb has been support TimeWindowCompactionStrategy, I use the newest scylladb with --experiment 1, it works well, and the performance is awesome |
sorry I found new issue "dependances" page is not support by scylladb, because of cassandra-spark-connector will use "ALLOW FILTERING", for performance reason, scylladb <= 2.3 is not supported, on scylladb offical website say it will support at 2.4. |
fyi we plan to remove that part of the schema in |
cool |
@yurishkuro Do you have an ETA when we can use scylla as a storage backend ? |
blocked in #793 |
Scylla Open Source 3.0 is currently nearing release; within the month. It will support ALLOW FILTERING. (Note: there will not likely be a Scylla Open Source Release 2.4; all references to 2.4 should refer to 3.0 instead.) |
SASI indices are removed by #1328 |
Who wants to give it a try and report back? :-) |
Scylla 3.0 with allow filtering and TWCS is there. We don't have SASI, is this a must? |
The requirement for SASI has been relaxed in 1.10, you can opt in to a data model that does not use SASI (will become the default going forward). Are there other incompatibilities? |
I have tested latest Jaeger 1.11 with scylla 3.0.4 and all things working fine.
|
@varunpalekar if you have that as a blog post, we could certainly help spread the word about it! |
@jpkrohling I have added my code in github repo https://github.com/varunpalekar/jaeger-with-ScyllaDB |
To add a datapoint: I have just setup jaeger 1.14.0 using scylla 3.1.1 as a backend without issue. We are currently ingesting ~5k spans a second but want to up that. If I do run across any problems I will report them here. |
We continue to see success with Scylla. I will warn anyone interested in using Jaeger with Scylla of this issue: scylladb/scylladb#5168. Jaeger uses TTL expiry and a TimeWindowCompactionStrategy as reported in the issue. It is not impacting us in 3.1.1, but it appears to impact versions at least as new as 3.0.10. |
@joe-elliott scylladb/scylladb#5168 is closed/fixed |
|
This is discoverable from the tacking issue, but there are no plans for official support, so closing. |
It seems a perfect big data solution in APM area.
The text was updated successfully, but these errors were encountered: