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

save space when union-ing DistinctTimestamps

Browse source 
By using an empty struct, we reduce memory usage by
up to 16 bytes (the size of a timestamp string)
This commit is contained in:
reesporte 2022-04-11 09:34:06 -05:00
parent f5c3c44411
commit 96df2700fd
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 {