Skip to content

Reenable MIRI check on PRs#421

Merged
alamb merged 1 commit into
apache:masterfrom
alamb:alamb/re-enable-miri
Jun 9, 2021
Merged

Reenable MIRI check on PRs#421
alamb merged 1 commit into
apache:masterfrom
alamb:alamb/re-enable-miri

Conversation

@alamb

@alamb alamb commented Jun 7, 2021

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Fixes: #345

Rationale:

We disabled this check in #360 as it was failing intermittently. However, @roee88 diagnosed the problem with a few tests that consumed a lot of memory, and @jhorstmann has been fixing some other MIR related issues in #416

Changes:

  1. Reenable MIRI tests
  2. Mark bit_length_test_string and bit_length_test_large_string to not run under MIRI
  3. Put them in their own github actions workflow (so that if they fail we don't have to rerun the entire workflow again)

@alamb

alamb commented Jun 7, 2021

Copy link
Copy Markdown
Contributor Author

Successful run 1: https://github.com/apache/arrow-rs/pull/421/checks?check_run_id=2765560493

@alamb

alamb commented Jun 8, 2021

Copy link
Copy Markdown
Contributor Author

@alamb alamb force-pushed the alamb/re-enable-miri branch from 984a88a to 503efb9 Compare June 8, 2021 20:49
@alamb alamb marked this pull request as ready for review June 8, 2021 20:50

@jorgecarleitao jorgecarleitao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@alamb

alamb commented Jun 8, 2021

Copy link
Copy Markdown
Contributor Author

@alamb

alamb commented Jun 9, 2021

Copy link
Copy Markdown
Contributor Author

Fourth success: https://github.com/apache/arrow-rs/pull/421/checks?check_run_id=2778308761

I am merging this one in :)

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.

MIRI CI check fails intermittently with thread 'main' panicked at 'invalid time'

2 participants