Skip to content

Reduce batch write tests run time#9553

Merged
kokosing merged 1 commit intotrinodb:masterfrom
wendigo:serafin/reduce-test-size
Oct 8, 2021
Merged

Reduce batch write tests run time#9553
kokosing merged 1 commit intotrinodb:masterfrom
wendigo:serafin/reduce-test-size

Conversation

@wendigo
Copy link
Contributor

@wendigo wendigo commented Oct 7, 2021

No description provided.

@wendigo wendigo requested a review from findepi October 7, 2021 20:09
@cla-bot cla-bot bot added the cla-signed label Oct 7, 2021
@wendigo wendigo requested a review from losipiuk October 7, 2021 20:13
@findepi findepi requested review from hashhar and kokosing October 7, 2021 20:14
{1000, 5000},
{10000, 1000},
{10000, 5000},
{10000, 15000},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in #8434 did you consider using smaller numbers?
@sergey-melnychuk @kokosing

Copy link
Member

@kokosing kokosing Oct 8, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we were not considering it as issue.

@wendigo wendigo requested a review from ebyhr October 8, 2021 07:00
{1000, 5000},
{10000, 1000},
{10000, 5000},
{10000, 15000},
Copy link
Member

@kokosing kokosing Oct 8, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we were not considering it as issue.

@kokosing kokosing merged commit 3fce161 into trinodb:master Oct 8, 2021
@kokosing
Copy link
Member

kokosing commented Oct 8, 2021

Merged, thanks!

@github-actions github-actions bot added this to the 364 milestone Oct 8, 2021
@wendigo wendigo deleted the serafin/reduce-test-size branch October 8, 2021 09:27
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