jorisvandenbossche commented on issue #39440: URL: https://github.com/apache/arrow/issues/39440#issuecomment-1881374547
Nice to have fixed given that it is a segfault, but it's not new, so I wouldn't say it's a blocker. We have this issue in general, when calling cython methods as class instead of instance methods, many of them will segfault. For example also: ``` In [9]: pa.Array.get_total_buffer_size(1) Segmentation fault (core dumped) ``` Because we nowhere do any type checking on `self` and just assume it is the correct type. Now, in this case of `ParquetFileFormat.make_write_options`, it's more important I think, because it makes more sense to indeed try to call this thinking it is a static class method. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
