Antoine Pitrou comments

Results 823 comments of


                                            Antoine Pitrou

Memory leak with Numpy arrays and the threaded scheduler

By the way, an orthogonal possibility would be for Anaconda to link its Python builds with an alternative malloc library such as [jemalloc](http://jemalloc.net/). Perhaps that would improve the numbers, or...

GH-43949: [C++] io::BufferedInput: Fixing the invalid state with SetBufferSize

Are you calling `SetBufferSize` from your own code? I think it was a mistake to make this API public, it should only be used for initializing the buffered stream.

GH-43949: [C++] io::BufferedInput: Fixing the invalid state with SetBufferSize

By the way, is this rebased on git main already?

GH-43017: [C++] Make the set of casts and hash kernels involving float16 consistent with other floating types

@felipecrv Do we have tests for sorting float16 values and NaNs already? Otherwise, we should add some. Also, is only `array_sort` handled, or does the more general `sort` also accept...

GH-43017: [C++] Make the set of casts and hash kernels involving float16 consistent with other floating types

@felipecrv Is this ready for review again?

GH-43046: [C++] Fix avx2 gather rows more than 2^31 issue in `CompareColumnsToRows`

Two things: 1) Yes, I agree we should fix the bug even though we can't write a unit test to avoid regressions 2) Are you saying the current fix is...

GH-43046: [C++] Fix avx2 gather rows more than 2^31 issue in `CompareColumnsToRows`

@zanmato1984 Sorry for the delay, I was away sick

GH-43046: [C++] Fix avx2 gather rows more than 2^31 issue in `CompareColumnsToRows`

@zanmato1984 Also, can you rebase?

GH-43046: [C++] Fix avx2 gather rows more than 2^31 issue in `CompareColumnsToRows`

> Also, if you think it's better to file the other issue ahead of this PR and have the TODO pointing to it, I'll do that. Thank you! Yes, that...

GH-32276: [C++][FlightRPC] Align RecordBatch buffers given to IPC

By the way, do we want to make this option an `enum` to allow for more options in the future? For example: ```c++ enum Alignment { // "natural" or "native",...