Skip to content

Conversation

@Manisha4
Copy link
Collaborator

@Manisha4 Manisha4 commented Jan 7, 2026

What this PR does / why we need it:

Switch from using ScyllaDB driver to Cassandra driver

Which issue(s) this PR fixes:

Misc

@Manisha4 Manisha4 changed the title Switch ScyllaDB driver to Cassandra driver feat: Switch ScyllaDB driver to Cassandra driver Jan 7, 2026
Copy link
Collaborator

@piket piket left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment on lines -194 to +198
"eg-feast[aws, azure, clickhouse, couchbase, delta, docling, duckdb, eg_valkey, elasticsearch, faiss, gcp, ge, go, grpcio, hazelcast, hbase, ibis, ikv, image, k8s, mcp, milvus, eg_milvus, mssql, mysql, opentelemetry, scylladb, spark, trino, postgres, pytorch, qdrant, rag, ray, redis, singlestore, snowflake, sqlite_vec]"
"eg-feast[aws, azure, clickhouse, couchbase, delta, docling, duckdb, eg_valkey, elasticsearch, faiss, gcp, ge, go, grpcio, hazelcast, hbase, ibis, ikv, image, k8s, mcp, milvus, eg_milvus, mssql, mysql, opentelemetry, cassandra, spark, trino, postgres, pytorch, qdrant, rag, ray, redis, singlestore, snowflake, sqlite_vec]"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We would need to verify that reads/writes with ScyllaDB as the online store would still work after this

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This section only defines the ci extra dependencies. It would still be possible to swap between scylla and cassandra by installing those extras explicitly.

@Manisha4 Manisha4 merged commit e90cc44 into stage-cassandra-switch Jan 8, 2026
38 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants