Skip to content

ARROW-15788: [C++][FlightRPC] Prepare benchmark for alternative transports - #12524

Closed
lidavidm wants to merge 1 commit into
apache:masterfrom
lidavidm:arrow-15788
Closed

ARROW-15788: [C++][FlightRPC] Prepare benchmark for alternative transports#12524
lidavidm wants to merge 1 commit into
apache:masterfrom
lidavidm:arrow-15788

Conversation

@lidavidm

@lidavidmlidavidm commented Feb 28, 2022

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actionsBot commented Feb 28, 2022

Copy link
Copy Markdown

https://issues.apache.org/jira/browse/ARROW-15788

@lidavidmlidavidm mentioned this pull request Mar 8, 2022
Closed
@lidavidm
lidavidm marked this pull request as ready for review March 14, 2022 18:21
@lidavidm

lidavidm commented Mar 14, 2022

Copy link
Copy Markdown
MemberAuthor

CC @cyb70289 if you'd like to take a look, this is just a minor refactoring to minimize the amount of churn that will be in #12442

cyb70289
cyb70289 approved these changes Mar 15, 2022
View reviewed changes

@cyb70289cyb70289 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@lidavidm
lidavidm deleted the arrow-15788 branch March 15, 2022 13:31
@ursabot

ursabot commented Mar 15, 2022•
edited
Loading

Copy link
Copy Markdown

Benchmark runs are scheduled for baseline = 98cdb73 and contender = b203c66. b203c66 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%] test-mac-arm
[Finished ⬇️0.71% ⬆️0.0%] ursa-i9-9960x
[Finished ⬇️0.04% ⬆️0.04%] ursa-thinkcentre-m75q
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java

@asfimportasfimport mentioned this pull request Mar 15, 2022
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cyb70289cyb70289cyb70289 approved these changes

Assignees

No one assigned

Labels

Component: C++Component: FlightRPC

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lidavidm@ursabot@cyb70289