Skip to content

Commit 5095312

Browse files
committed
changefeedccl: import proto schema from changefeedpb repo
This change updates dependencies and imports the proto schema from the external changefeedpb repository. Fixes: #149703
1 parent 853c27e commit 5095312

File tree

15 files changed

+56
-231
lines changed

15 files changed

+56
-231
lines changed

BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@ exports_files([
5454
# gazelle:resolve go github.com/cockroachdb/cockroach/pkg/sql/colflow_test //pkg/sql/colflow:colflow_test
5555
# gazelle:resolve go github.com/cockroachdb/cockroach/pkg/util/caller_test //pkg/util/caller:caller_test
5656
# gazelle:resolve go github.com/cockroachdb/cockroach/pkg/util/json_test //pkg/util/json:json_test
57+
# gazelle:resolve go github.com/cockroachdb/changefeedpb @com_github_cockroachdb_changefeedpb//:go_default_library
5758
# gazelle:resolve go google.golang.org/genproto/googleapis/pubsub/v1 @org_golang_google_genproto//googleapis/pubsub/v1:pubsub
5859
# gazelle:resolve go google.golang.org/genproto/googleapis/cloud/kms/v1 @org_golang_google_genproto//googleapis/cloud/kms/v1:kms
5960
# gazelle:resolve go google.golang.org/genproto/googleapis/devtools/build/v1 @org_golang_google_genproto//googleapis/devtools/build/v1:build

DEPS.bzl

Lines changed: 20 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1721,6 +1721,21 @@ def go_deps():
17211721
"https://storage.googleapis.com/cockroach-godeps/gomod/github.com/cockroachdb/apd/v3/com_github_cockroachdb_apd_v3-v3.2.1.zip",
17221722
],
17231723
)
1724+
go_repository(
1725+
name = "com_github_cockroachdb_changefeedpb",
1726+
build_directives = [
1727+
"gazelle:go_proto_compilers @com_github_cockroachdb_cockroach//pkg/cmd/protoc-gen-gogoroach:protoc-gen-gogoroach_compiler",
1728+
"gazelle:go_grpc_compilers @com_github_cockroachdb_cockroach//pkg/cmd/protoc-gen-gogoroach:protoc-gen-gogoroach_grpc_compiler",
1729+
"gazelle:resolve go go github.com/golang/protobuf/ptypes/timestamp @com_github_golang_protobuf//ptypes/timestamp:go_default_library",
1730+
],
1731+
build_file_proto_mode = "default",
1732+
importpath = "github.com/cockroachdb/changefeedpb",
1733+
sha256 = "94533028aec2a4a56b8c5e6ea0e43513dcc160075765cbc6fbe720a0fcafc6a5",
1734+
strip_prefix = "github.com/cockroachdb/[email protected]",
1735+
urls = [
1736+
"https://storage.googleapis.com/cockroach-godeps/gomod/github.com/cockroachdb/changefeedpb/com_github_cockroachdb_changefeedpb-v0.0.0-20250807204158-c4f634c8d95c.zip",
1737+
],
1738+
)
17241739
go_repository(
17251740
name = "com_github_cockroachdb_cmux",
17261741
build_file_proto_mode = "disable_global",
@@ -1774,10 +1789,10 @@ def go_deps():
17741789
go_repository(
17751790
name = "com_github_cockroachdb_errors",
17761791
build_directives = [
1777-
"gazelle:resolve proto proto gogoproto/gogo.proto @com_github_gogo_protobuf//gogoproto:gogo_proto",
1778-
"gazelle:resolve proto go gogoproto/gogo.proto @com_github_gogo_protobuf//gogoproto",
17791792
"gazelle:go_proto_compilers @com_github_cockroachdb_cockroach//pkg/cmd/protoc-gen-gogoroach:protoc-gen-gogoroach_compiler",
1780-
"gazelle:go_grpc_compilers @com_github_cockroachdb_cockroach//pkg/cmd/protoc-gen-gogoroach:protoc-gen-gogoroach_grpc_compiler",
1793+
"gazelle:go_grpc_compilers @com_github_cockroachdb_cockroach//pkg/cmd/protoc-gen-gogoroach:protoc-gen-gogoroach_grpc_compiler",
1794+
"gazelle:resolve proto proto gogoproto/gogo.proto @com_github_gogo_protobuf//gogoproto:gogo_proto",
1795+
"gazelle:resolve proto go gogoproto/gogo.proto @com_github_gogo_protobuf//gogoproto",
17811796
],
17821797
build_file_proto_mode = "default",
17831798
importpath = "github.com/cockroachdb/errors",
@@ -7548,6 +7563,8 @@ def go_deps():
75487563
go_repository(
75497564
name = "com_github_prometheus_client_model",
75507565
build_directives = [
7566+
"gazelle:go_proto_compilers @com_github_cockroachdb_cockroach//pkg/cmd/protoc-gen-gogoroach:protoc-gen-gogoroach_compiler",
7567+
"gazelle:go_grpc_compilers @com_github_cockroachdb_cockroach//pkg/cmd/protoc-gen-gogoroach:protoc-gen-gogoroach_grpc_compiler",
75517568
"gazelle:resolve go go github.com/golang/protobuf/ptypes/timestamp @com_github_golang_protobuf//ptypes/timestamp:go_default_library",
75527569
],
75537570
build_file_proto_mode = "default",

build/bazelutil/distdir_files.bzl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -344,6 +344,7 @@ DISTDIR_FILES = {
344344
"https://storage.googleapis.com/cockroach-godeps/gomod/github.com/cockroachdb/apd/com_github_cockroachdb_apd-v1.1.0.zip": "fef7ec2fae220f84bfacb17fbfc1b04a666ab7f6fc04f3ff6d2b1e05c380777d",
345345
"https://storage.googleapis.com/cockroach-godeps/gomod/github.com/cockroachdb/apd/v3/com_github_cockroachdb_apd_v3-v3.2.1.zip": "6ad54bb71a36fba8ca6725a00d916e51815a4c68de54096313ca6fffda6c87c2",
346346
"https://storage.googleapis.com/cockroach-godeps/gomod/github.com/cockroachdb/bubbletea/com_github_cockroachdb_bubbletea-v0.23.1-bracketed-paste2.zip": "d7916a0e7d8d814566e8f8d162c3764aea947296396a0a669564ff3ee53414bc",
347+
"https://storage.googleapis.com/cockroach-godeps/gomod/github.com/cockroachdb/changefeedpb/com_github_cockroachdb_changefeedpb-v0.0.0-20250807204158-c4f634c8d95c.zip": "94533028aec2a4a56b8c5e6ea0e43513dcc160075765cbc6fbe720a0fcafc6a5",
347348
"https://storage.googleapis.com/cockroach-godeps/gomod/github.com/cockroachdb/client_golang/com_github_cockroachdb_client_golang-v0.0.0-20250124161916-2d4b7d300341.zip": "b4fc2d09aab49c3315b442db09b08bbc7f164d5536404443cbb57203d1cd461f",
348349
"https://storage.googleapis.com/cockroach-godeps/gomod/github.com/cockroachdb/cmux/com_github_cockroachdb_cmux-v0.0.0-20250514152509-914d3bf9ec58.zip": "c1cf4cd99a1ad6a00f2ccd4188cbcf004cb0d56895670b2c171061ce564cd791",
349350
"https://storage.googleapis.com/cockroach-godeps/gomod/github.com/cockroachdb/cockroach-go/v2/com_github_cockroachdb_cockroach_go_v2-v2.4.1.zip": "ba646db91152f3121a6812c7b74d12d8c0e126f7b4d3b927618b159692ceb424",

go.mod

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,7 @@ require (
128128
github.com/blevesearch/snowballstem v0.9.0
129129
github.com/charmbracelet/bubbles v0.15.1-0.20230123181021-a6a12c4a31eb
130130
github.com/cockroachdb/apd/v3 v3.2.1
131+
github.com/cockroachdb/changefeedpb v0.0.0-20250807204158-c4f634c8d95c
131132
github.com/cockroachdb/cmux v0.0.0-20250514152509-914d3bf9ec58
132133
github.com/cockroachdb/cockroach-go/v2 v2.4.1
133134
github.com/cockroachdb/crlib v0.0.0-20250718215705-7ff5051265b9

go.sum

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -549,6 +549,8 @@ github.com/cockroachdb/apd/v3 v3.2.1 h1:U+8j7t0axsIgvQUqthuNm82HIrYXodOV2iWLWtEa
549549
github.com/cockroachdb/apd/v3 v3.2.1/go.mod h1:klXJcjp+FffLTHlhIG69tezTDvdP065naDsHzKhYSqc=
550550
github.com/cockroachdb/bubbletea v0.23.1-bracketed-paste2 h1:OmQDBlTg1DU5OaKjIuEBt4K+40i9RqtxAwK1P/0OVjI=
551551
github.com/cockroachdb/bubbletea v0.23.1-bracketed-paste2/go.mod h1:JAfGK/3/pPKHTnAS8JIE2u9f61BjWTQY57RbT25aMXU=
552+
github.com/cockroachdb/changefeedpb v0.0.0-20250807204158-c4f634c8d95c h1:92Sd4x3IQbEiOZQ9vUo+cwWS6m/saC53tofAknPBWBA=
553+
github.com/cockroachdb/changefeedpb v0.0.0-20250807204158-c4f634c8d95c/go.mod h1:Czh2+FFDUUKdsxZ7aYZSW0W/E0Dx2sQXf1UUkeMkzxw=
552554
github.com/cockroachdb/client_golang v0.0.0-20250124161916-2d4b7d300341 h1:VQDZhIT4rPz6OhMeiD/2kXfE2PLHJzX2QUd1wsGdjxM=
553555
github.com/cockroachdb/client_golang v0.0.0-20250124161916-2d4b7d300341/go.mod h1:Zsulrv/L9oM40tJ7T815tM89lFEugiJ9HzIqaAx4LKc=
554556
github.com/cockroachdb/cmux v0.0.0-20250514152509-914d3bf9ec58 h1:DQM99rWou5NZoKKKgSFUtO10FmeK8jbetLIdaH2LHE8=

pkg/ccl/changefeedccl/BUILD.bazel

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -162,6 +162,7 @@ go_library(
162162
"//pkg/util/unique",
163163
"//pkg/util/uuid",
164164
"@com_github_apache_pulsar_client_go//pulsar",
165+
"@com_github_cockroachdb_changefeedpb//:go_default_library",
165166
"@com_github_cockroachdb_errors//:errors",
166167
"@com_github_cockroachdb_logtags//:logtags",
167168
"@com_github_cockroachdb_redact//:redact",
@@ -354,6 +355,7 @@ go_test(
354355
"//pkg/workload/workloadsql",
355356
"@com_github_apache_pulsar_client_go//pulsar",
356357
"@com_github_cockroachdb_apd_v3//:apd",
358+
"@com_github_cockroachdb_changefeedpb//:go_default_library",
357359
"@com_github_cockroachdb_cockroach_go_v2//crdb",
358360
"@com_github_cockroachdb_errors//:errors",
359361
"@com_github_cockroachdb_redact//:redact",

pkg/ccl/changefeedccl/changefeedpb/BUILD.bazel

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,9 @@ load("@io_bazel_rules_go//proto:def.bzl", "go_proto_library")
44

55
proto_library(
66
name = "changefeedpb_proto",
7-
srcs = [
8-
"changefeed.proto",
9-
"scheduled_changefeed.proto",
10-
],
7+
srcs = ["scheduled_changefeed.proto"],
118
strip_import_prefix = "/pkg",
129
visibility = ["//visibility:public"],
13-
deps = ["@com_google_protobuf//:timestamp_proto"],
1410
)
1511

1612
go_proto_library(

pkg/ccl/changefeedccl/changefeedpb/changefeed.proto

Lines changed: 0 additions & 154 deletions
This file was deleted.

pkg/ccl/changefeedccl/changefeedpb/marshal.go

Lines changed: 0 additions & 55 deletions
Original file line numberDiff line numberDiff line change
@@ -40,58 +40,3 @@ func (m ScheduledChangefeedExecutionArgs) MarshalJSONPB(x *jsonpb.Marshaler) ([]
4040
m.ChangefeedStatement = tree.AsStringWithFlags(export, tree.FmtShowFullURIs)
4141
return json.Marshal(m)
4242
}
43-
44-
// MarshalJSON provides clean, unwrapped JSON output for changefeedpb.Value.
45-
func (v *Value) MarshalJSON() ([]byte, error) {
46-
if v == nil || v.Value == nil {
47-
return []byte("null"), nil
48-
}
49-
50-
var out any
51-
switch val := v.Value.(type) {
52-
case *Value_BoolValue:
53-
out = val.BoolValue
54-
case *Value_Int64Value:
55-
out = val.Int64Value
56-
case *Value_DoubleValue:
57-
out = val.DoubleValue
58-
case *Value_StringValue:
59-
out = val.StringValue
60-
case *Value_DecimalValue:
61-
out = val.DecimalValue.Value
62-
case *Value_TimestampValue:
63-
out = val.TimestampValue
64-
case *Value_DateValue:
65-
out = val.DateValue
66-
case *Value_IntervalValue:
67-
out = val.IntervalValue
68-
case *Value_UuidValue:
69-
out = val.UuidValue
70-
case *Value_BytesValue:
71-
out = string(val.BytesValue)
72-
case *Value_TimeValue:
73-
out = val.TimeValue
74-
case *Value_ArrayValue:
75-
arr := make([]any, len(val.ArrayValue.Values))
76-
for i, elem := range val.ArrayValue.Values {
77-
arr[i] = elem
78-
}
79-
out = arr
80-
case *Value_TupleValue:
81-
m := make(map[string]any, len(val.TupleValue.Values))
82-
for k, v := range val.TupleValue.Values {
83-
m[k] = v
84-
}
85-
out = m
86-
default:
87-
return nil, errors.AssertionFailedf("unexpected protobuf value type: %T", v.Value)
88-
}
89-
return json.Marshal(out)
90-
}
91-
92-
func (k *Key) MarshalJSON() ([]byte, error) {
93-
if k == nil || k.Key == nil {
94-
return []byte("null"), nil
95-
}
96-
return json.Marshal(k.Key)
97-
}

pkg/ccl/changefeedccl/encoder_protobuf.go

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,9 @@ import (
1010
"fmt"
1111
"time"
1212

13+
"github.com/cockroachdb/changefeedpb"
1314
"github.com/cockroachdb/cockroach/pkg/ccl/changefeedccl/cdcevent"
1415
"github.com/cockroachdb/cockroach/pkg/ccl/changefeedccl/changefeedbase"
15-
"github.com/cockroachdb/cockroach/pkg/ccl/changefeedccl/changefeedpb"
1616
"github.com/cockroachdb/cockroach/pkg/sql/sem/tree"
1717
"github.com/cockroachdb/cockroach/pkg/sql/sessiondatapb"
1818
"github.com/cockroachdb/cockroach/pkg/util/hlc"
@@ -395,13 +395,22 @@ func datumToProtoValue(
395395
if err != nil {
396396
return nil, err
397397
}
398-
return &changefeedpb.Value{Value: &changefeedpb.Value_TimestampValue{TimestampValue: ts}}, nil
398+
return &changefeedpb.Value{
399+
Value: &changefeedpb.Value_TimestampValue{
400+
TimestampValue: ts,
401+
},
402+
}, nil
403+
399404
case *tree.DTimestamp:
400405
ts, err := types.TimestampProto(v.Time.UTC())
401406
if err != nil {
402407
return nil, err
403408
}
404-
return &changefeedpb.Value{Value: &changefeedpb.Value_TimestampValue{TimestampValue: ts}}, nil
409+
return &changefeedpb.Value{
410+
Value: &changefeedpb.Value_TimestampValue{
411+
TimestampValue: ts,
412+
},
413+
}, nil
405414
case *tree.DBytes:
406415
return &changefeedpb.Value{Value: &changefeedpb.Value_BytesValue{BytesValue: []byte(*v)}}, nil
407416
case *tree.DGeography:

0 commit comments

Comments
 (0)