Skip to content

PARQUET-1980: Add TravisCI job to build and test on ARM64#167

Closed
martin-g wants to merge 1 commit intoapache:masterfrom
martin-g:master
Closed

PARQUET-1980: Add TravisCI job to build and test on ARM64#167
martin-g wants to merge 1 commit intoapache:masterfrom
martin-g:master

Conversation

@martin-g
Copy link
Copy Markdown
Member

Jira

https://issues.apache.org/jira/browse/PARQUET-1980

Commits

  • [ X ] 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

  • No new functionality

- sudo make install
- cd ..

script: mvn --batch-mode --show-version clean verify
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This just makes the script task explicit.

@gszadovszky
Copy link
Copy Markdown
Contributor

gszadovszky commented Feb 16, 2021

@martin-g, because of the recent Travis issue we have decided to move to Github Actions. We've already done with the transition in parquet-mr and it is on the way for parquet-format as well. See #166 for details.

Another thing is that the parquet-format builds are only for verification purposes. The generated java classes are deprecated here and will be removed in a later release. We have moved the java related code generation to the parquet-mr module parquet-format-structures.

@martin-g
Copy link
Copy Markdown
Member Author

Thanks for the reference, @gszadovszky !
I've added a comment to PARQUET-1969

@martin-g
Copy link
Copy Markdown
Member Author

Closing this PR since the code is now in Parquet MR!

@martin-g martin-g closed this Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants