Skip to content

Partial revert of increase split scheduling config#19877

Merged
sopel39 merged 1 commit intotrinodb:masterfrom
Dith3r:ke/splits-revert
Nov 27, 2023
Merged

Partial revert of increase split scheduling config#19877
sopel39 merged 1 commit intotrinodb:masterfrom
Dith3r:ke/splits-revert

Conversation

@Dith3r
Copy link
Member

@Dith3r Dith3r commented Nov 23, 2023

Description

Partially revert #19487, reduce regressions in tpch q18/q21

Additional context and related issues

Release notes

(X) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Nov 23, 2023
@Dith3r Dith3r requested review from losipiuk and sopel39 November 23, 2023 18:03
@Dith3r Dith3r force-pushed the ke/splits-revert branch 2 times, most recently from 5ed3cc9 to 263d675 Compare November 23, 2023 19:37
@losipiuk
Copy link
Member

Do you have benchmarks like in original PR to see if partial revert does degrade some other cases?

@losipiuk
Copy link
Member

nit: some release notes might be useful if we can describe what is changed in general terms; not refering to specific tpch query numbers.

@Dith3r Dith3r closed this Nov 24, 2023
@Dith3r Dith3r reopened this Nov 24, 2023
@sopel39 sopel39 merged commit b2efce7 into trinodb:master Nov 27, 2023
@sopel39 sopel39 mentioned this pull request Nov 27, 2023
@github-actions github-actions bot added this to the 434 milestone Nov 27, 2023
@Dith3r Dith3r deleted the ke/splits-revert branch February 21, 2024 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants