Make the hash map group by lazy

[joka921]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 92.75362% with 5 lines in your changes missing coverage. Please review.

Project coverage is 89.87%. Comparing base (97c195a) to head (896df47).

Files with missing lines	Patch %	Lines
src/engine/GroupBy.cpp	92.75%	0 Missing and 5 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1651      +/-   ##
==========================================
+ Coverage   89.84%   89.87%   +0.02%     
==========================================
  Files         389      389              
  Lines       37268    37287      +19     
  Branches     4202     4204       +2     
==========================================
+ Hits        33482    33510      +28     
+ Misses       2485     2478       -7     
+ Partials     1301     1299       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[hannahbast]

Looks good and works fine in some test queries we tried. Minor change left.

[hannahbast]

Thanks!

As a follow-up, we should find a way to enable this automatically, in particular, for queries like SELECT ?g (COUNT(?g) AS ?count) WHERE { GRAPH ?g { ?s ?p ?o } } GROUP BY ?g

[sparql-conformance]

Conformance check passed ✅

No test result changes.

Details: https://qlever.cs.uni-freiburg.de/sparql-conformance-ui?cur=896df47f275e3d8d2a3ac3c381e504c3920c2004&prev=97c195aaf9e249f43a6417f99ea93f7e94831f39

[sonarqubecloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud