Comments (5)
cc @cockroachdb/cdc
from cockroach.
Hi @wenyihu6, please add branch-* labels to identify which branch(es) this GA-blocker affects.
🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.
from cockroach.
@wenyihu6 Is another possible fix here to change this loop:
cockroach/pkg/ccl/changefeedccl/changefeed_processors.go
Lines 528 to 533 in df19639
to make sure that it returns 0 when there is a 0 timestamp anywhere in the watches? I haven't read carefully, but it seems to me that loop is incorrect in the case where, say ca.spec.Watches
contains some series of spans with an empty timestamp, but then the last span has a non-empty timestamp.
We had a similar bug in PCR: #110085
from cockroach.
I agree that this could be another possible fix as far as I can tell. But I still want to read the code more to see how it plays along with other changefeeds and backfill since this is legacy code from
cockroach/pkg/ccl/changefeedccl/changefeed_processors.go
Lines 418 to 422 in 0eda540
from cockroach.
Reopening since we've found that although disabling shutdown checkpointing / drain watcher reduces the likelihood of hitting this issue, this issue pre-exists that feature.
from cockroach.
Related Issues (20)
- kvserver: allow configuring the max gossip peers
- internal/sqlsmith: TestRandTableInserts failed HOT 1
- kv: manual campaigning should respect fortification HOT 1
- kv: benchmark use of leader leases on simple workloads HOT 1
- tsdump: add flag that lets you specify additional resolutions HOT 7
- raft: add StoreLiveness interface HOT 2
- kv: remove COCKROACH_DISABLE_LEADER_FOLLOWS_LEASEHOLDER HOT 1
- kv: test switching back and forth between leader and epoch leases HOT 1
- raft: add MsgFortify and MsgFortifyResp HOT 1
- roachtest: unoptimized-query-oracle/disable-rules=half/rand-tables failed HOT 1
- raft: add concept of quorum supported expiration HOT 1
- raft: add SupportTracker HOT 1
- raft: refactor ProgressTracker to support leader leases HOT 1
- raft: make commit index delivery in MsgApp reliable HOT 1
- sqlstats: disable collecting statement stats by transaction fingerprint id
- pkg/ccl/backupccl/backuprand/backuprand_test: TestBackupRestoreRandomDataRoundtrips failed HOT 1
- storage: TestNoMinVerFile failed
- : failed
- pkg/compose/compose_test: TestComposeCompare failed
- roachtest: lease-preferences/manual-violating-transfer failed HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from cockroach.