This is an automated email from the ASF dual-hosted git repository.
zeroshade pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-go.git
The following commit(s) were added to refs/heads/main by this push:
new 460f500 chore: Bump github.com/substrait-io/substrait-go/v3 from
3.6.0 to 3.7.0 (#287)
460f500 is described below
commit 460f5004b92d3cb84dde89d56d11340619adab15
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
AuthorDate: Tue Feb 18 16:51:00 2025 -0500
chore: Bump github.com/substrait-io/substrait-go/v3 from 3.6.0 to 3.7.0
(#287)
Bumps
[github.com/substrait-io/substrait-go/v3](https://github.com/substrait-io/substrait-go)
from 3.6.0 to 3.7.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/substrait-io/substrait-go/releases">github.com/substrait-io/substrait-go/v3's
releases</a>.</em></p>
<blockquote>
<h1>v3.7.0 (2025-02-16)</h1>
<h3>Features</h3>
<ul>
<li>add support for NewDecimalFromApdDecimal (<a
href="https://redirect.github.com/substrait-io/substrait-go/issues/119">#119</a>)
(<a
href="https://github.com/substrait-io/substrait-go/commit/e64c78608a0cae9552e35bd4fea3efbe75e83049">e64c786</a>)</li>
<li>load the default extension collection lazily (<a
href="https://redirect.github.com/substrait-io/substrait-go/issues/118">#118</a>)
(<a
href="https://github.com/substrait-io/substrait-go/commit/261dc94fc7c25ace64696e96e1f839f05075e200">261dc94</a>)</li>
<li>Add AggregateRel.ToBuilder() (<a
href="https://redirect.github.com/substrait-io/substrait-go/issues/114">#114</a>)
(<a
href="https://github.com/substrait-io/substrait-go/commit/3be7ba38b3cae32415fa528bd22859feee820677">3be7ba3</a>)</li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/substrait-io/substrait-go/commit/e64c78608a0cae9552e35bd4fea3efbe75e83049"><code>e64c786</code></a>
feat: add support for NewDecimalFromApdDecimal (<a
href="https://redirect.github.com/substrait-io/substrait-go/issues/119">#119</a>)</li>
<li><a
href="https://github.com/substrait-io/substrait-go/commit/261dc94fc7c25ace64696e96e1f839f05075e200"><code>261dc94</code></a>
feat: load the default extension collection lazily (<a
href="https://redirect.github.com/substrait-io/substrait-go/issues/118">#118</a>)</li>
<li><a
href="https://github.com/substrait-io/substrait-go/commit/3be7ba38b3cae32415fa528bd22859feee820677"><code>3be7ba3</code></a>
feat: Add AggregateRel.ToBuilder() (<a
href="https://redirect.github.com/substrait-io/substrait-go/issues/114">#114</a>)</li>
<li>See full diff in <a
href="https://github.com/substrait-io/substrait-go/compare/v3.6.0...v3.7.0">compare
view</a></li>
</ul>
</details>
<br />
[](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)
</details>
---------
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot]
<49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Matt Topol <[email protected]>
---
arrow/compute/exprs/builders.go | 3 ++-
arrow/compute/exprs/exec.go | 5 +++--
go.mod | 2 +-
go.sum | 4 ++--
4 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/arrow/compute/exprs/builders.go b/arrow/compute/exprs/builders.go
index 5868e7a..71be97f 100644
--- a/arrow/compute/exprs/builders.go
+++ b/arrow/compute/exprs/builders.go
@@ -36,7 +36,8 @@ import (
// Arrow Extension registry and the default collection of substrait extensions
// from the Substrait-go repo.
func NewDefaultExtensionSet() ExtensionIDSet {
- return
NewExtensionSetDefault(expr.NewEmptyExtensionRegistry(&extensions.DefaultCollection))
+ return NewExtensionSetDefault(
+
expr.NewEmptyExtensionRegistry(extensions.GetDefaultCollectionWithNoError()))
}
// NewScalarCall constructs a substrait ScalarFunction expression with the
provided
diff --git a/arrow/compute/exprs/exec.go b/arrow/compute/exprs/exec.go
index 1a51f2e..2e64381 100644
--- a/arrow/compute/exprs/exec.go
+++ b/arrow/compute/exprs/exec.go
@@ -173,7 +173,7 @@ func GetExtensionIDSet(ctx context.Context) ExtensionIDSet {
v, ok := ctx.Value(extCtxKey{}).(ExtensionIDSet)
if !ok {
return NewExtensionSet(
-
expr.NewEmptyExtensionRegistry(&extensions.DefaultCollection),
+
expr.NewEmptyExtensionRegistry(extensions.GetDefaultCollectionWithNoError()),
GetExtensionRegistry(ctx))
}
return v
@@ -434,7 +434,8 @@ func ExecuteScalarSubstrait(ctx context.Context, expression
*expr.Extended, part
}
reg := GetExtensionRegistry(ctx)
- set := NewExtensionSet(expr.NewExtensionRegistry(expression.Extensions,
&extensions.DefaultCollection), reg)
+ set := NewExtensionSet(expr.NewExtensionRegistry(expression.Extensions,
+ extensions.GetDefaultCollectionWithNoError()), reg)
sc, err := ToArrowSchema(expression.BaseSchema, set)
if err != nil {
return nil, err
diff --git a/go.mod b/go.mod
index b2ae89b..976b71d 100644
--- a/go.mod
+++ b/go.mod
@@ -39,7 +39,7 @@ require (
github.com/pterm/pterm v0.12.80
github.com/stoewer/go-strcase v1.3.0
github.com/stretchr/testify v1.10.0
- github.com/substrait-io/substrait-go/v3 v3.6.0
+ github.com/substrait-io/substrait-go/v3 v3.7.0
github.com/tidwall/sjson v1.2.5
github.com/zeebo/xxh3 v1.0.2
golang.org/x/exp v0.0.0-20240909161429-701f63a606c0
diff --git a/go.sum b/go.sum
index 4825388..cfd5e5a 100644
--- a/go.sum
+++ b/go.sum
@@ -164,8 +164,8 @@ github.com/stretchr/testify v1.10.0
h1:Xv5erBjTwe/5IxqUQTdXv5kgmIvbHo3QQyRwhJsOf
github.com/stretchr/testify v1.10.0/go.mod
h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
github.com/substrait-io/substrait v0.66.1-0.20250205013839-a30b3e2d7ec6
h1:XqtxwYFCjS4L0o1QD4ipGHCuFG94U0f6BeldbilGQjU=
github.com/substrait-io/substrait v0.66.1-0.20250205013839-a30b3e2d7ec6/go.mod
h1:MPFNw6sToJgpD5Z2rj0rQrdP/Oq8HG7Z2t3CAEHtkHw=
-github.com/substrait-io/substrait-go/v3 v3.6.0
h1:t+mb4WskugKDVRqAZvV6Y0m34AQcDrd0cZ3NOXDcUjU=
-github.com/substrait-io/substrait-go/v3 v3.6.0/go.mod
h1:VG7jCqtUm28bSngHwq86FywtU74knJ25LNX63SZ53+E=
+github.com/substrait-io/substrait-go/v3 v3.7.0
h1:rBxnPXTtxgPJTLpMvZgcEXuVyzmO0SkM2EpEUEvdN5s=
+github.com/substrait-io/substrait-go/v3 v3.7.0/go.mod
h1:VG7jCqtUm28bSngHwq86FywtU74knJ25LNX63SZ53+E=
github.com/tidwall/gjson v1.14.2
h1:6BBkirS0rAHjumnjHF6qgy5d2YAJ1TLIaFE2lzfOLqo=
github.com/tidwall/gjson v1.14.2/go.mod
h1:/wbyibRr2FHMks5tjHJ5F8dMZh3AcwJEMf5vlfC0lxk=
github.com/tidwall/match v1.1.1 h1:+Ho715JplO36QYgwN9PGYNhgZvoUSc9X2c80KVTi+GA=