Skip to content

ARROW-14261: [C++] Includes should be in alphabetical order#11362

Closed
bkmgit wants to merge 3 commits into
apache:masterfrom
bkmgit:ARROW-14261
Closed

ARROW-14261: [C++] Includes should be in alphabetical order#11362
bkmgit wants to merge 3 commits into
apache:masterfrom
bkmgit:ARROW-14261

Conversation

@bkmgit

@bkmgit bkmgit commented Oct 8, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Oct 8, 2021

Copy link
Copy Markdown

@kou

kou commented Oct 8, 2021

Copy link
Copy Markdown
Member

@github-actions autotune

@cyb70289

cyb70289 commented Oct 9, 2021

Copy link
Copy Markdown
Contributor

MacOS CI error is not related.

@cyb70289 cyb70289 closed this in 39875f1 Oct 9, 2021
@ursabot

ursabot commented Oct 9, 2021

Copy link
Copy Markdown

Benchmark runs are scheduled for baseline = 1b0ea15 and contender = 39875f1. 39875f1 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Finished ⬇️0.0% ⬆️0.0%] ec2-t3-xlarge-us-east-2
[Finished ⬇️0.0% ⬆️0.0%] ursa-i9-9960x
[Finished ⬇️0.0% ⬆️0.0%] ursa-thinkcentre-m75q
Supported benchmarks:
ursa-i9-9960x: langs = Python, R, JavaScript
ursa-thinkcentre-m75q: langs = C++, Java
ec2-t3-xlarge-us-east-2: cloud = True

@cyb70289

cyb70289 commented Oct 9, 2021

Copy link
Copy Markdown
Contributor

Thanks @bkmgit

@bkmgit

bkmgit commented Oct 9, 2021

Copy link
Copy Markdown
Contributor Author

Thanks for the reviews @kou @cyb70289

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants