Skip to content
Snippets Groups Projects

F files

Merged Timm Fitschen requested to merge f-files into dev
3 files
+ 15
16
Compare changes
  • Side-by-side
  • Inline

Files

+ 2
13
@@ -253,17 +253,6 @@ 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()`
*/
@@ -391,7 +380,7 @@ public:
* this transaction. In all other cases, the return value will be
* -1.
*/
[[nodiscard]] inline auto GetCountResult() const -> int {
[[nodiscard]] inline auto GetCountResult() const -> long {
return query_count;
}
@@ -406,7 +395,7 @@ public:
}
/**
* Get a JSON representation of the respone.
* Get a JSON representation of the response.
*
* For debugging.
*/
Loading