storage: Fix testdrive test to not overwrite schema #22054
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is correcting a testdrive test which inadvertently worked.
So, apparently it's possible to create multiple sinks on the same kafka topic and we just publish a new avro schema? I guess we missed the test was using the same topic to publish different schema and the overwrite wasn't taking long. With some refactoring in #21852, the CI started continuously failing https://buildkite.com/materialize/tests/builds/64722#018add21-f1a5-4258-bf3a-01148d0e33a6.
Motivation
Tips for reviewer
Checklist
$T ⇔ Proto$T
mapping (possibly in a backwards-incompatible way), then it is tagged with aT-proto
label.