-
Notifications
You must be signed in to change notification settings - Fork 4k
Closed
Description
google/snappy#148 was finally merged, so if we upgrade to the current HEAD, we will resolve the UBSan issues we've observed and can remove some test skips (in R at least). And I see google/snappy@8dd58a5 as well, which may let us remove the ARROW_SNAPPY_OLD_BUILD_VERSION workaround too.
We could wait for an official snappy release, but they seem infrequent. Last one was over a year ago and there have been 2 sanitizer fixes we've upstreamed since then.
Reporter: Neal Richardson / @nealrichardson
Related issues:
- [R] [CI] Clean up our snappy-sanitizer skipping behavior (is related to)
Note: This issue was originally created as ARROW-17283. Please see the migration documentation for further details.