diff --git a/include/caosdb/transaction.h b/include/caosdb/transaction.h index ee848469cf6fd23abde4542c8a1db7f12ac94c76..26982cdcf62c1d065bc0432c5823d2c6e02ba1e6 100644 --- a/include/caosdb/transaction.h +++ b/include/caosdb/transaction.h @@ -249,6 +249,17 @@ private: std::unique_ptr<Entity> entity; }; +// TODO(fspreck) One possibility for count-queries: Transaction gets a +// GetQueryCount function that returns exactly this. In case of a +// single FIND Query this will be equivalent to the Size of the +// ResultSet, but it may be different in case of Query + RetrieveById, +// and in case of COUNT queries which have an empty ResultSet. We have +// to think about how to extend this to transactions with more than +// one query in the future. My suggestion is to treat a transaction +// with more than one query as the union of the query results in which +// case the query count would return the cumulative number of +// results. This would even allow the combination of FIND, FIND +// unique, and COUNT, albeit a little strangely. /** * @brief Create a transaction via `CaosDBConnection.createTransaction()` */