Grouping error about sqlpp11 HOT 4 CLOSED

Hi,

Letting this sink in for a while, it does not seem to be too complicated after all:

Lets say that aggregate expressions are either

• calls to aggregate functions like count or avg or
• group by expressions or
• combinations of aggregate expressions

Then

• combining aggregate expressions and non-aggregate expressions (e.g. myAggregate + myNonAggregate) is not allowed
• either all result columns have to be aggregate expressions or all result columns have to be non-aggregate expressions.

Since aggregate functions are known via their tag types, and group by can yield a set of expressions, this should not be too difficult to check.

If a database (like MySQL) supports mixing aggregate and non-aggregate columns, then this can be supported by the connector of that database offering something like sqlpp::mysql::pseudo_aggregate (better name, anyone?), a no-op function that just marks a column as aggregate.

Best,

Roland

from sqlpp11.

This should be fixed by now in the develop branch. The develop branch requires a minor change in the connector libraries as well. See the latest changes to connection.h in the sqlite3 connector. Let me know if you need help adjusting the postgresql connector.

from sqlpp11.

Hi.
I Will handle this, after proper testing of the "returning" feature. If I need help, I'll let you know

from sqlpp11.

As far as I can tell, this is fixed in release-0.35.

It was not trivial, though. Please let me know if I missed something.

from sqlpp11.