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

GH-44062: [Dev][Archery][Integration] Reduce needless test matrix #44099

Merged
merged 8 commits into from
Sep 16, 2024

Commits on Sep 13, 2024

  1. apacheGH-44062: [Dev][Archery][Integration] Reduce needless test matrix

    If we enable C++, Java and Rust, we use the following patterns:
    
    | Producer | Consumer |
    |----------|----------|
    | C++      | C++      |
    | C++      | Java     |
    | C++      | Rust     |
    | Java     | C++      |
    | Java     | Java     |
    | Java     | Rust     |
    | Rust     | C++      |
    | Rust     | Java     |
    | Rust     | Rust     |
    
    In apache/arrow, the following patterns are redundant because they
    should be done in apache/arrow-rs:
    
    | Producer | Consumer |
    |----------|----------|
    | Rust     | Rust     |
    
    In apache/arror-rs, the following patterns are redundant because they
    should be done in apache/arrow:
    
    | Producer | Consumer |
    |----------|----------|
    | C++      | C++      |
    | C++      | Java     |
    | Java     | C++      |
    | Java     | Java     |
    
    Add `--target-language` option. We can specify target languages by
    this. (We can specify `--target-language` multiple times.) Here are
    expected usages:
    
    In apache/arrow:
    * `--target-language=cpp`
    * `--target-language=csharp`
    * `--target-language=go`
    * `--target-language=java`
    * `--target-language=js`
    
    In apache/arrow-rs
    * `--target-language=rust`
    
    Here is an example in apache/arrow-rs:
    
    T: Languages specified by `--target-language`
       * rust
    O: Languages not specified by `--target-language`
       * cpp
       * csharp
       * go
       * java
       * js
       * nanoarrow
    
    Used matrix:
    
    | Producer | Consumer |
    |----------|----------|
    | Rust     | Rust     |
    | Rust     | C++      |
    | Rust     | C#       |
    | Rust     | Go       |
    | Rust     | Java     |
    | Rust     | JS       |
    | Rust     | nanoarrow|
    | C++      | Rust     |
    | C#       | Rust     |
    | Go       | Rust     |
    | Java     | Rust     |
    | JS       | Rust     |
    | nanoarrow| Rust     |
    kou committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    80551c3 View commit details
    Browse the repository at this point in the history
  2. Use string

    kou committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    bad8c35 View commit details
    Browse the repository at this point in the history
  3. Fix a typo

    kou committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    f39ed69 View commit details
    Browse the repository at this point in the history
  4. Fix lint

    kou committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    1055eb3 View commit details
    Browse the repository at this point in the history
  5. Fix empty string case

    kou committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    0567f1a View commit details
    Browse the repository at this point in the history
  6. Fix a typo

    kou committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    b092bc3 View commit details
    Browse the repository at this point in the history

Commits on Sep 14, 2024

  1. Add a debug print

    kou committed Sep 14, 2024
    Configuration menu
    Copy the full SHA
    b3c9cb7 View commit details
    Browse the repository at this point in the history
  2. Add missing export

    kou committed Sep 14, 2024
    Configuration menu
    Copy the full SHA
    f4fd547 View commit details
    Browse the repository at this point in the history