roachtest: cdc/initial-scan-only failed [NPE in btree-based frontier] #115411
Assignees
Labels
A-cdc
Change Data Capture
branch-master
Failures and bugs on the master branch.
C-bug
Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
C-test-failure
Broken test (automatically or manually discovered).
O-roachtest
O-robot
Originated from a bot.
P-1
Issues/test failures with a fix SLA of 1 month
release-blocker
Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked.
T-cdc
Milestone
Comments
cockroach-teamcity
added
branch-master
stevendanna
changed the title
|
This appears to be a NPE in the new btree-based frontier: Similar errors are happening on other roachtests that make use of frontiers such as the C2C roachtests. |
This was referenced Dec 1, 2023
|
roachtest.cdc/initial-scan-only failed with artifacts on master @ 74ae9a18d82fdc80e5e9b71b3f06b258ae5cb91b: Parameters: |
miretskiy
pushed a commit
to miretskiy/cockroach
that referenced
this issue
Dec 3, 2023
Re-initialize iterator when forwarding span frontier timestamp. The underlying btree may be mutated (by merge operation) invalidating previously constructed iterator. Fixes cockroachdb#115411 Release notes: None
craig bot
pushed a commit
that referenced
this issue
Dec 4, 2023
113952: log: add protobuf messages for telemetry txn events r=xinhaoz a=xinhaoz This commit adds the messages listed below to `telemetry.proto` in preparation for sending transaction executions to the telemetry channel. The transaction event that is eventually sent should contain all execution information currently being tracked for transaction fingerprints. - `SampledTransaction`: contains fields equivalent to the execution information stored by `CollectedTransactionStatistics` from app_stats.proto, but represents a single txn execution instead of aggregated executions of a transaction fingerprint. - `SampledExecStats`: used as a field in `SampledTransaction`, it contains execution stats that are sampled. This event is the equivalent to `ExecStats` from app_stats.proto but for a single execution. - `MVCCIteratorStats`: used in `SampledExecStats` above, the equivalent of MVCCIteratorStats from app_stats.proto but for a single execution. In addition, in order to support the above fields a couple of additional code templates have been added for generating json log encoding: - array_of_uint64 type is now being handled for json logs - `nestedMessage` has been added as a custom type in `gen.go`. Object field types can be assigned to this type in order to generate them as nested objects. Part of: #108284 Release note: None 114666: opt: reduce planning time for queries with many joins r=mgartner a=mgartner Prior to this commit, some queries with many joins would perform a large number of allocations calculating the selectivity of null-rejecting join filters. This was due to `statisticsBuiler.selectivityFromNullsRemoved` allocating a single-column set for each not-null column, and allocating column statistics for each set. Many of those allocations and much unnecessary computations to traverse the expression tree are now avoided. This is made possible by the realization that the selectivity of a null-rejecting filter is always 1 if the column was already not-null in the input. Epic: None Release note: None 115509: span: Re-initialize iterator when forwarding r=miretskiy a=miretskiy Re-initialize iterator when forwarding span frontier timestamp. The underlying btree may be mutated (by merge operation) invalidating previously constructed iterator. Btree implementation is also hardened against mis-use when mutating span frontier while iterating. Fixes #115411 Fixes #115528 Fixes #115512 Fixes #115490 Fixes #115488 Fixes #115487 Fixes #115483 Release notes: None Co-authored-by: Xin Hao Zhang <[email protected]> Co-authored-by: Marcus Gartner <[email protected]> Co-authored-by: Yevgeniy Miretskiy <[email protected]>
miretskiy
pushed a commit
to miretskiy/cockroach
that referenced
this issue
Dec 4, 2023
Re-initialize iterator when forwarding span frontier timestamp. The underlying btree may be mutated (by merge operation) invalidating previously constructed iterator. Fixes cockroachdb#115411 Release notes: None
This was referenced Dec 5, 2023
This was referenced Dec 5, 2023
rharding6373
added
the
C-bug
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
roachtest.cdc/initial-scan-only failed with artifacts on master @ 02e46d54d0b0bf63f43592709d551534edb54be6:
Parameters:
ROACHTEST_arch=amd64,ROACHTEST_cloud=gce,ROACHTEST_cpu=16,ROACHTEST_encrypted=false,ROACHTEST_fs=ext4,ROACHTEST_localSSD=true,ROACHTEST_metamorphicBuild=false,ROACHTEST_ssd=0Help
See: roachtest README
See: How To Investigate (internal)
See: Grafana
This test on roachdash | Improve this report!
Jira issue: CRDB-34017
The text was updated successfully, but these errors were encountered: