Skip to content

PARQUET-2205: Accelerate CI by commonizing thrift build and leveraging cache#1007

Open
sekikn wants to merge 1 commit into
apache:masterfrom
sekikn:PARQUET-2205-PR
Open

PARQUET-2205: Accelerate CI by commonizing thrift build and leveraging cache#1007
sekikn wants to merge 1 commit into
apache:masterfrom
sekikn:PARQUET-2205-PR

Conversation

@sekikn

@sekikn sekikn commented Oct 26, 2022

Copy link
Copy Markdown
Contributor

Make sure you have checked all steps below.

Jira

Tests

  • My PR adds the following unit tests OR does not need testing for this extremely good reason:

This fix changes only CI-related files, so it's enough to make sure that CI jobs are improved.

Commits

  • My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":
    1. Subject is separated from body by a blank line
    2. Subject is limited to 50 characters (not including Jira issue reference)
    3. Subject does not end with a period
    4. Subject uses the imperative mood ("add", not "adding")
    5. Body wraps at 72 characters
    6. Body explains "what" and "why", not "how"

Documentation

  • In case of new functionality, my PR adds documentation that describes how to use it.
    • All the public functions and the classes in the PR contain Javadoc that explain what it does

@sekikn

sekikn commented Oct 26, 2022

Copy link
Copy Markdown
Contributor Author

CI usually takes over 30 minutes on my GitHub account. With this PR, commonizing thrift build improved the total duration to 28.5 minutes, and if cache exists, the total duration was improved to ~26 minutes.

@github-actions

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has had no activity for at least 2 months. If you are still working on this change or plan to move it forward, please leave a comment or push a new commit so we know to keep it open. Otherwise, this PR will be closed automatically in about one month. Thank you for your contribution to Apache Parquet!

@github-actions github-actions Bot added the stale label Jun 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant