nevi-me commented on pull request #8200:
URL: https://github.com/apache/arrow/pull/8200#issuecomment-724906422
@jorgecarleitao finally passed! There's still some tests that don't pass,
but I've updated the status documenation with them (cc @andygrove as you had
opened a JIRA for this)
nevi-me commented on pull request #8200:
URL: https://github.com/apache/arrow/pull/8200#issuecomment-724504342
@jorgecarleitao we're down to 3 failures.
```rust
thread 'main' panicked at 'index out of bounds: the len is 0 but the index
is 0',
nevi-me commented on pull request #8200:
URL: https://github.com/apache/arrow/pull/8200#issuecomment-724493404
> From what I have read in this PR, it looks good so far. Is there anything
blocking it?
@jorgecarleitao the actual integration tests not passing, are the blocker
nevi-me commented on pull request #8200:
URL: https://github.com/apache/arrow/pull/8200#issuecomment-724489474
@jorgecarleitao @carols10cents I'm contemplating closing this, and breaking
it up into smaller PRs (if this round of changes don't pass integration). I'll
work on it this week in
nevi-me commented on pull request #8200:
URL: https://github.com/apache/arrow/pull/8200#issuecomment-706553767
> Thus, IMO `capacity` main use-case is for bookkeeping, on how to
de-allocate the region. #8401 systematizes that idea, on which `BufferData`
(renamed `Bytes` there) no longer
