Skip to content
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

Set 'task.partitioned-writer-count' property to processor count in testExplicitPropertyMappings test #15010

Merged
merged 1 commit into from
Nov 14, 2022

Conversation

sanchitkashyap
Copy link
Member

In the current state, the test can sometimes fail depending on the number of cores of the machine in which the test is running.

@cla-bot
Copy link

cla-bot bot commented Nov 14, 2022

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Sanchit Kashyap.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@cla-bot cla-bot bot added the cla-signed label Nov 14, 2022
@findepi findepi requested review from raunaqmorarka and sopel39 and removed request for findepi November 14, 2022 13:02
@sopel39
Copy link
Member

sopel39 commented Nov 14, 2022

Thanks
@sanchitkashyap please also file CLA

@sopel39 sopel39 merged commit e82edd9 into trinodb:master Nov 14, 2022
@github-actions github-actions bot added this to the 403 milestone Nov 14, 2022
@colebow colebow added the no-release-notes This pull request does not require release notes entry label Nov 15, 2022
@sanchitkashyap sanchitkashyap deleted the fix_unitTest branch November 15, 2022 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed no-release-notes This pull request does not require release notes entry
Development

Successfully merging this pull request may close these issues.

4 participants