Matt Jaffee 16fd6a7edd add tests for GroupBy(Distinct), fix various problems ... We execute the aggregate Distinct calls after the GroupBy is complete, and we need these to act like non-remote calls in that they forward to all nodes, but like remote calls in that they bypass key translation. Added a "PreTranslated" flag to the QueryRequest to achieve this. Discovered an issue where a nil *Row in EmbeddedData would cause a panic in the protobuf serialization. Changed the encoding code we control to never pass a nil *Row. Got fed up with lack of context on errors and added wrapping to all calls under executor.executeCall as well as a few other places. Handled a situation where not having data on a shard for a particular field could cause a query to error instead of just treating that fragment as being empty. (see the switch in executeDistinctShardSet) Stopped GroupBy from executing the Count(Distinct) aggregate on Remote calls. Fixed a longstanding issue where errors retrieved from remote query calls had a garbage character at the front due to treating a protobuf payload as an error message instead of decoding it. (see http/client.go)		2020-12-30 08:13:41 -06:00
..
proto.go	add tests for GroupBy(Distinct), fix various problems	2020-12-30 08:13:41 -06:00