Skip to content

Enable Arm builds#12451

Merged
abhishekagarwal87 merged 16 commits intoapache:masterfrom
2bethere:graviton2
Apr 26, 2022
Merged

Enable Arm builds#12451
abhishekagarwal87 merged 16 commits intoapache:masterfrom
2bethere:graviton2

Conversation

@2bethere
Copy link
Copy Markdown
Contributor

@2bethere 2bethere commented Apr 18, 2022

Enable ARM builds

Description

This PR enables ARM builds on Travis. I've ported over the changes from @martin-g on reducing heap requirements for some of the tests to ensure they run well on Travis arm instances.


This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

martin-g and others added 16 commits April 20, 2021 09:37
... because at the moment they took 1h which is a little bit above the TravisCI limit of 50mins per job and because @clintropolis requested to add one more module - sql
... because at the moment they took 1h which is a little bit above the TravisCI limit of 50mins per job and because @clintropolis requested to add one more module - sql
@2bethere
Copy link
Copy Markdown
Contributor Author

Seems the build failure are all related to flaky tests. (e.g. docker hub not available, zk timeouts)

@martin-g this is effectively your PR and I'd like to help push this forward. Any other concerns or things you'd like to add before I start try to get review on this one?

@2bethere 2bethere changed the title Testing out arm builds Enable Arm builds Apr 20, 2022
@martin-g
Copy link
Copy Markdown
Member

@2bethere Everything looks good to me! Thank you for doing this!

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.

5 participants