fdb-record-layer
fdb-record-layer copied to clipboard
Remove OnlineIndexer functions that take Key.Evaluateds
The online indexing API defines a few places where it takes a Key.Evaluated
(and also usually defines an overload where it takes a Tuple
). Most of the API, however, uses Tuple
s as primary keys, so that seems like the more logical way to specify key ranges (and also will save some conversions to and from Tuple
s and Key.Evaluated
s).
This is, of course, a breaking change for those who are using the API. I think we can remove all internal usage and then deprecate those methods...probably.
@alecgrieser, is that (yet) an open issue?
I'm not sure I understand the question. I think this is still an open issue, unless someone removed the Key.Evaluated
methods or we decided that we do want to keep them, for some reason