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.

Sign up for GitHub

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

Jump to bottom

ARROW-15984: [C++] Change RecordBatchReader API to use Result<> #12678

Closed
wants to merge 13 commits into from
Closed

ARROW-15984: [C++] Change RecordBatchReader API to use Result<> #12678

wants to merge 13 commits into from

Commits on Mar 23, 2022

  1. change RecordBatchReader API to use Result<>

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    d5d5a55 View commit details
    Browse the repository at this point in the history

  2. gtest_util: introduce ASSERT_OK_NO_THROW_AND_ASSIGN

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    6beb5cb View commit details
    Browse the repository at this point in the history

  3. update usages of RecordBatchReader::ReadAll

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    2413eb5 View commit details
    Browse the repository at this point in the history

  4. add tests for deprecated RecordBatchReader::ReadAll 

    regualar tests no longer cover this method
    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    f2df172 View commit details
    Browse the repository at this point in the history

  5. fix formatting

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    25e1490 View commit details
    Browse the repository at this point in the history

  6. more formatting fix

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    16c4bc8 View commit details
    Browse the repository at this point in the history

  7. update r bindings

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    e0d9133 View commit details
    Browse the repository at this point in the history

  8. tests for non-deprecated ToTable/ToRecordBatches

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    27eab58 View commit details
    Browse the repository at this point in the history

  9. c_glib: update ReadAll usage

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    f3e96c2 View commit details
    Browse the repository at this point in the history

  10. update ReadAll usage in pyarrow

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    2261297 View commit details
    Browse the repository at this point in the history

  11. remove ASSERT_OK_NO_THROW_AND_ASSIGN 

    ASSERT_OK_NO_THROW use was unnecessary in the first place
    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    f75e137 View commit details
    Browse the repository at this point in the history

  12. fix mixup betwee deprecated and non deprecated API tests

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    ae9137a View commit details
    Browse the repository at this point in the history

  13. update remaining pyarrow ReadAll use

    @zagto
    zagto committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    b69e257 View commit details
    Browse the repository at this point in the history