vitrivr / cottontaildb-proto Goto Github PK

Ideally, the SimpleClient should have the functionality to set the queryID which is currently used for logging in Cottontail.

Currently, the CreateIndex message simply takes an indexName as parameter. That name is however expected to be formatted including schema and entity name. It would be better to explicitly add those parameters to the index-creation message.

Some return types for methods of the CottonDDL service are not optimal:

• ListSchemas returns a single schema and not multiple
• CreateSchema and DropSchema don't return a status regarding the operations
• CreateEntity returns a single entity rather than multiple
• DropEntity, OptimizeEntity and TruncateEntity don't return a status regarding the operations

Currently, if you're trying to create an entity which already exists or trying to drop an entity which not exists (among other things), cottontaildb throws a StatusRuntimeException over the channel. This is because the .proto-definition does not allow for transparent failure.

One solution would be to include a Success Flag in the SuccessStatus Message or return a StatusMessage which can include information about unsuccessfull operations without throwing exceptions.

An InsertMessage has a list of Tuples which in turn has a map of Data. All vectors are capsuled in a Vector message, which can not be part of a Tuple.