-
Notifications
You must be signed in to change notification settings - Fork 2.5k
[Dev] Remove upsert_conflict_in_different_chunk.test
#15980
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Dev] Remove upsert_conflict_in_different_chunk.test
#15980
Conversation
Thanks to Sam we narrowed it down to |
@Mytherin I'm not exactly sure what the right course of action here is |
Skipping the test seems fine to me - although I wonder if we want to test this behavior in the first place? Do we want to make an effort to ensure this guarantee holds up (if |
I don't feel strongly about it either way
When executed in parallel this falls apart, but otherwise I think we can guarantee it? |
FORCE_ASYNC_SINK_SOURCE
upsert_conflict_in_different_chunk.test
Thanks! |
[Dev] Fix issue related to unpacked columns and the NOT operator (duckdb/duckdb#15534) [Dev] Remove `upsert_conflict_in_different_chunk.test` (duckdb/duckdb#15980)
[Dev] Fix issue related to unpacked columns and the NOT operator (duckdb/duckdb#15534) [Dev] Remove `upsert_conflict_in_different_chunk.test` (duckdb/duckdb#15980)
[Dev] Fix issue related to unpacked columns and the NOT operator (duckdb/duckdb#15534) [Dev] Remove `upsert_conflict_in_different_chunk.test` (duckdb/duckdb#15980)
[Dev] Fix issue related to unpacked columns and the NOT operator (duckdb/duckdb#15534) [Dev] Remove `upsert_conflict_in_different_chunk.test` (duckdb/duckdb#15980)
[Dev] Fix issue related to unpacked columns and the NOT operator (duckdb/duckdb#15534) [Dev] Remove `upsert_conflict_in_different_chunk.test` (duckdb/duckdb#15980)
[Dev] Fix issue related to unpacked columns and the NOT operator (duckdb/duckdb#15534) [Dev] Remove `upsert_conflict_in_different_chunk.test` (duckdb/duckdb#15980)
This PR fixes https://github.com/duckdblabs/duckdb-internal/issues/3919