search

foundationdb-beam / ecto_foundationdb

FoundationDB Adapter for Elixir's Ecto
Apache License 2.0
15 stars 1 forks source link

Remove required use of tenants #18

Open JesseStimpson opened 6 days ago

JesseStimpson commented 6 days ago

The FoundationDB project does not fully support tenants, and the feature is unlikely to lose its "experimental" status in the near future. (Based on discussion in apple/foundationdb#11292 and apple/foundationdb#11382)

EctoFDB should not require users to use tenants. We'll keep the feature available, and we'll continue to use it where it makes sense (integration tests), but we should discourage its use in production.

JesseStimpson commented 6 days ago

Maybe we can use the directory layer to expose multitenancy instead of using FDB Tenants.

JesseStimpson commented 5 days ago

Using the directory layer doesn't appear to be compatible with get_mapped_range due to the fact that the (startByteIndex:stopByteIndex) syntax on the mapper's element selector is not yet supported.

Everything about GetMappedRange

So for now, it appears that we need to choose between "directory layer tenants" and "get mapped range". Since everything appears to be functional with FDB tenants so far, and get_mapped_range is a valuable feature, for now we'll keep FDB Tenants, but we'll have to update the documentation to make the risks clear.