Skip to content

Commit a6c0a5e

Browse files
benbellickclaude
andauthored
chore: Bump github.com/substrait-io/substrait-go from v7.6.0 to v8.0.0 (#754)
### Rationale for this change Upgrade substrait-go to the latest major version (v8). I'm visiting from @substrait-io and want to bump this so I can update the dependency in `iceberg-go`. ### What changes are included in this PR? - Updated `github.com/substrait-io/substrait-go` from v7.6.0 to v8.0.0 - Updated import paths from `v7` to `v8` across `arrow/compute/exprs` - Transitive dependency bumps: `substrait-protobuf/go` v0.84.0 → v0.85.0, `substrait` v0.84.0 → v0.85.0 ### Are these changes tested? Existing tests pass (`go test ./arrow/compute/exprs/...`). ### Are there any user-facing changes? Users importing `arrow/compute/exprs` will transitively depend on substrait-go v8 instead of v7. Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 3c8d9c6 commit a6c0a5e

8 files changed

Lines changed: 22 additions & 22 deletions

File tree

arrow/compute/exprs/builders.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,9 @@ import (
2727

2828
"github.com/apache/arrow-go/v18/arrow"
2929
"github.com/apache/arrow-go/v18/arrow/compute"
30-
"github.com/substrait-io/substrait-go/v7/expr"
31-
"github.com/substrait-io/substrait-go/v7/extensions"
32-
"github.com/substrait-io/substrait-go/v7/types"
30+
"github.com/substrait-io/substrait-go/v8/expr"
31+
"github.com/substrait-io/substrait-go/v8/extensions"
32+
"github.com/substrait-io/substrait-go/v8/types"
3333
)
3434

3535
// NewDefaultExtensionSet constructs an empty extension set using the default

arrow/compute/exprs/builders_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ import (
2525
"github.com/apache/arrow-go/v18/arrow/compute/exprs"
2626
"github.com/stretchr/testify/assert"
2727
"github.com/stretchr/testify/require"
28-
"github.com/substrait-io/substrait-go/v7/expr"
28+
"github.com/substrait-io/substrait-go/v8/expr"
2929
)
3030

3131
func TestNewScalarFunc(t *testing.T) {

arrow/compute/exprs/exec.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,9 @@ import (
3232
"github.com/apache/arrow-go/v18/arrow/internal/debug"
3333
"github.com/apache/arrow-go/v18/arrow/memory"
3434
"github.com/apache/arrow-go/v18/arrow/scalar"
35-
"github.com/substrait-io/substrait-go/v7/expr"
36-
"github.com/substrait-io/substrait-go/v7/extensions"
37-
"github.com/substrait-io/substrait-go/v7/types"
35+
"github.com/substrait-io/substrait-go/v8/expr"
36+
"github.com/substrait-io/substrait-go/v8/extensions"
37+
"github.com/substrait-io/substrait-go/v8/types"
3838
)
3939

4040
func makeExecBatch(ctx context.Context, schema *arrow.Schema, partial compute.Datum) (out compute.ExecBatch, err error) {

arrow/compute/exprs/exec_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ import (
3434
"github.com/google/uuid"
3535
"github.com/stretchr/testify/assert"
3636
"github.com/stretchr/testify/require"
37-
"github.com/substrait-io/substrait-go/v7/expr"
38-
"github.com/substrait-io/substrait-go/v7/types"
37+
"github.com/substrait-io/substrait-go/v8/expr"
38+
"github.com/substrait-io/substrait-go/v8/types"
3939
proto "github.com/substrait-io/substrait-protobuf/go/substraitpb"
4040
)
4141

arrow/compute/exprs/field_refs.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ import (
2626
"github.com/apache/arrow-go/v18/arrow/compute"
2727
"github.com/apache/arrow-go/v18/arrow/memory"
2828
"github.com/apache/arrow-go/v18/arrow/scalar"
29-
"github.com/substrait-io/substrait-go/v7/expr"
29+
"github.com/substrait-io/substrait-go/v8/expr"
3030
)
3131

3232
func getFields(typ arrow.DataType) []arrow.Field {

arrow/compute/exprs/types.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,9 @@ import (
2828
"github.com/apache/arrow-go/v18/arrow"
2929
"github.com/apache/arrow-go/v18/arrow/compute"
3030
"github.com/apache/arrow-go/v18/arrow/scalar"
31-
"github.com/substrait-io/substrait-go/v7/expr"
32-
"github.com/substrait-io/substrait-go/v7/extensions"
33-
"github.com/substrait-io/substrait-go/v7/types"
31+
"github.com/substrait-io/substrait-go/v8/expr"
32+
"github.com/substrait-io/substrait-go/v8/extensions"
33+
"github.com/substrait-io/substrait-go/v8/types"
3434
)
3535

3636
const (

go.mod

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ require (
3737
github.com/pterm/pterm v0.12.83
3838
github.com/stoewer/go-strcase v1.3.1
3939
github.com/stretchr/testify v1.11.1
40-
github.com/substrait-io/substrait-go/v7 v7.6.0
41-
github.com/substrait-io/substrait-protobuf/go v0.84.0
40+
github.com/substrait-io/substrait-go/v8 v8.0.0
41+
github.com/substrait-io/substrait-protobuf/go v0.85.0
4242
github.com/tidwall/sjson v1.2.5
4343
github.com/zeebo/xxh3 v1.1.0
4444
golang.org/x/exp v0.0.0-20260112195511-716be5621a96
@@ -78,7 +78,7 @@ require (
7878
github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 // indirect
7979
github.com/remyoudompheng/bigfft v0.0.0-20230129092748-24d4a6f8daec // indirect
8080
github.com/stretchr/objx v0.5.2 // indirect
81-
github.com/substrait-io/substrait v0.84.0 // indirect
81+
github.com/substrait-io/substrait v0.85.0 // indirect
8282
github.com/tidwall/gjson v1.14.2 // indirect
8383
github.com/tidwall/match v1.1.1 // indirect
8484
github.com/tidwall/pretty v1.2.0 // indirect

go.sum

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -152,12 +152,12 @@ github.com/stretchr/testify v1.8.0/go.mod h1:yNjHg4UonilssWZ8iaSj1OCr/vHnekPRkoO
152152
github.com/stretchr/testify v1.8.1/go.mod h1:w2LPCIKwWwSfY2zedu0+kehJoqGctiVI29o6fzry7u4=
153153
github.com/stretchr/testify v1.11.1 h1:7s2iGBzp5EwR7/aIZr8ao5+dra3wiQyKjjFuvgVKu7U=
154154
github.com/stretchr/testify v1.11.1/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
155-
github.com/substrait-io/substrait v0.84.0 h1:krf3WFSltV184/JUJirwYlyR6ksgccVc3IAjIc9/ePM=
156-
github.com/substrait-io/substrait v0.84.0/go.mod h1:MPFNw6sToJgpD5Z2rj0rQrdP/Oq8HG7Z2t3CAEHtkHw=
157-
github.com/substrait-io/substrait-go/v7 v7.6.0 h1:YMo/ZS0XqHoNSvQ/TRxkQ03iE47vk0z+gl8LXCPazZM=
158-
github.com/substrait-io/substrait-go/v7 v7.6.0/go.mod h1:/THTJcGbArvo7tPHMUkSlWdQxJ9LED2WYrp5qNb9DhA=
159-
github.com/substrait-io/substrait-protobuf/go v0.84.0 h1:UcaZ+CE7l2UKJcNY9QlGcFFKv6h4jFDo8QhdTb5L4X0=
160-
github.com/substrait-io/substrait-protobuf/go v0.84.0/go.mod h1:hn+Szm1NmZZc91FwWK9EXD/lmuGBSRTJ5IvHhlG1YnQ=
155+
github.com/substrait-io/substrait v0.85.0 h1:ur2VBFhOpx/3RjVG0w5i8SpLHciLjATh27kAc2HPf5A=
156+
github.com/substrait-io/substrait v0.85.0/go.mod h1:MPFNw6sToJgpD5Z2rj0rQrdP/Oq8HG7Z2t3CAEHtkHw=
157+
github.com/substrait-io/substrait-go/v8 v8.0.0 h1:8of34zk+SNPV9BjBh1Bcm7G/Qk/2oMZl20DE0+SAWIg=
158+
github.com/substrait-io/substrait-go/v8 v8.0.0/go.mod h1:q98eLpY/OQgeH3d7cYBl2nJvoFAZxOyX9K13kpsKJ0Q=
159+
github.com/substrait-io/substrait-protobuf/go v0.85.0 h1:zk6MtNWLtDSl8a7qCZRFH0+EIIXVrrd/hsgYK/SQTgM=
160+
github.com/substrait-io/substrait-protobuf/go v0.85.0/go.mod h1:hn+Szm1NmZZc91FwWK9EXD/lmuGBSRTJ5IvHhlG1YnQ=
161161
github.com/tidwall/gjson v1.14.2 h1:6BBkirS0rAHjumnjHF6qgy5d2YAJ1TLIaFE2lzfOLqo=
162162
github.com/tidwall/gjson v1.14.2/go.mod h1:/wbyibRr2FHMks5tjHJ5F8dMZh3AcwJEMf5vlfC0lxk=
163163
github.com/tidwall/match v1.1.1 h1:+Ho715JplO36QYgwN9PGYNhgZvoUSc9X2c80KVTi+GA=

0 commit comments

Comments
 (0)