Skip to content

Commit

Permalink
Merge pull request #1517 fixed optional param in ydb.ParamsBuilder
Browse files Browse the repository at this point in the history
rekby authored Oct 25, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
2 parents 6afc9a7 + 19cea5b commit 7fc096e
Showing 4 changed files with 126 additions and 345 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
* Fixed send optional arguments to the server with `ydb.ParamsBuilder`

## v3.88.0
* Removed UUID methods from ydb.ParamsBuilder()

2 changes: 1 addition & 1 deletion internal/params/optional.go
Original file line number Diff line number Diff line change
@@ -23,7 +23,7 @@ func (b *optionalBuilder) EndOptional() Builder {
b.opt.parent.params = append(b.opt.parent.params, &Parameter{
parent: b.opt.parent,
name: b.opt.name,
value: value.OptionalValue(b.opt.value),
value: b.opt.value,
})

return b.opt.parent
Loading

0 comments on commit 7fc096e

Please sign in to comment.