Skip to content
This repository was archived by the owner on Sep 9, 2024. It is now read-only.

mock: validate empty partition keys for all operations #87

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
+77 −0
Open

mock: validate empty partition keys for all operations #87

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
6869e53
mock: validate empty partition keys for all operations
slai Mar 28, 2023
71858aa
clean up comment
slai Mar 28, 2023
8ebcaef
more comment tweaks
slai Mar 28, 2023
b5dc3a4
allow no partition keys
slai Mar 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
clean up comment
@slai
slai committed Mar 28, 2023
commit 71858aa9713f23e6471a98ab2c8d03b952afadf9
2 changes: 1 addition & 1 deletion multimap_multikey_table.go
View file
Original file line number Diff line number Diff line change
@@ -88,7 +88,7 @@ func (mm *multimapMkT) ListOfEqualRelations(fieldsToIndex, ids map[string]interf
}
}

// TODO: support empty values in composite clustering keys
// TODO: support empty values in clustering columns
for _, field := range mm.idField {
if value := ids[field]; value != nil && value != "" {
relation := Eq(field, value)