andishgar commented on PR #46625: URL: https://github.com/apache/arrow/pull/46625#issuecomment-2917074922
@kou I have several questions which is not related to our discussion. 1- Does Arrow allow to have an Array which has buffers from several Device or MemoryManager? 1.1- if the answer is no, Should we consider the following line as bug? https://github.com/apache/arrow/blob/b3e4d2244e1b2449efed54dec6e7c79405a029db/cpp/src/arrow/gpu/cuda_test.cc#L1059-L1068 2- The question is located in the comment below. https://github.com/apache/arrow/blob/b3e4d2244e1b2449efed54dec6e7c79405a029db/cpp/src/arrow/gpu/cuda_test.cc#L1192-L1194 3- The result of the following expression is false: `my_cuda_mm_->AllocateBuffer(10).ValueOrDie()->memory_manager() == my_cuda_mm_` 3.1- Is this the expected behavior? 3.2- If the answer is no, should we consider the above line a bug? https://github.com/apache/arrow/blob/b3e4d2244e1b2449efed54dec6e7c79405a029db/cpp/src/arrow/gpu/cuda_test.cc#L1237-L1240 4-` arrow::CudaHostBuffer::FromVector` and `arrow::CudaBuffer::FromVector `both call the following lines, which do not produce either a `arrow::CudaHostBuffer` or a` CudaBuffer` . Should this be considered a bug? https://github.com/apache/arrow/blob/b3e4d2244e1b2449efed54dec6e7c79405a029db/cpp/src/arrow/buffer.h#L162-L180 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
