Skip to content

GH-36618: [C++] Add a test for evaluation of ARROW_CHECK payload#36617

Merged
pitrou merged 1 commit into
apache:mainfrom
pitrou:arrow_check_evaluation_test
Jul 11, 2023
Merged

GH-36618: [C++] Add a test for evaluation of ARROW_CHECK payload#36617
pitrou merged 1 commit into
apache:mainfrom
pitrou:arrow_check_evaluation_test

Conversation

@pitrou

@pitrou pitrou commented Jul 11, 2023

Copy link
Copy Markdown
Member

We want to make sure that the error description payload for ARROW_CHECK is only evaluated if the check fails.

We want to make sure that the error description payload for ARROW_CHECK is only evaluated if the check fails.
@github-actions

Copy link
Copy Markdown

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

In the case of PARQUET issues on JIRA the title also supports:

PARQUET-${JIRA_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

See also:

@pitrou

pitrou commented Jul 11, 2023

Copy link
Copy Markdown
Member Author

@felipecrv

@pitrou pitrou changed the title [C++] Add a test for evaluation of ARROW_CHECK payload GH-36618: [C++] Add a test for evaluation of ARROW_CHECK payload Jul 11, 2023
@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #36618 has been automatically assigned in GitHub to PR creator.

@pitrou pitrou marked this pull request as ready for review July 11, 2023 15:57
@github-actions github-actions Bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Jul 11, 2023
@pitrou

pitrou commented Jul 11, 2023

Copy link
Copy Markdown
Member Author

Thanks for the review @felipecrv . CI failures are unrelated, will merge.

@pitrou pitrou merged commit 0e0df29 into apache:main Jul 11, 2023
@pitrou pitrou removed the awaiting committer review Awaiting committer review label Jul 11, 2023
@pitrou pitrou deleted the arrow_check_evaluation_test branch July 11, 2023 16:41
@conbench-apache-arrow

Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 6 benchmarking runs that have been run so far on merge-commit 0e0df29.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about possible false positives for unstable benchmarks that are known to sometimes produce them.

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.

[C++] Add a test for evaluation of ARROW_CHECK payload

2 participants