scriptoriumadmin/featurebase
1
0
Fork 0
mirror of https://github.com/featurebasedb/featurebase.git synced 2026-08-28 10:54:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #2015 from molecula/union-distinct-ts-better

Browse source 
save space when union-ing DistinctTimestamps
This commit is contained in:
reesporte 2022-04-14 11:11:36 -05:00 • committed by GitHub
commit e2bf96731f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
executor.go
View file

@ -1617,12 +1617,12 @@ func (d DistinctTimestamp) ToRows(callback func(*proto.RowResponse) error) error
// Union returns the union of the values of `d` and `other`
func (d *DistinctTimestamp) Union(other DistinctTimestamp) DistinctTimestamp {
both := map[string]string{}
both := map[string]struct{}{}
for _, val := range d.Values {
both[val] = val
both[val] = struct{}{}
}
for _, val := range other.Values {
both[val] = val
both[val] = struct{}{}
}
vals := []string{}
for key := range both {