alamb opened a new issue #221:
URL: https://github.com/apache/arrow-rs/issues/221


   *Note*: migrated from original JIRA: 
https://issues.apache.org/jira/browse/ARROW-10888
   
   We have a number of functions in the Rust code base that use unsafe code but 
they are not declared as unsafe or documented as to why we are guaranteeing 
that they are actually safe.
   
   I have seen recent criticism of the project on social media because of this 
and it is something that we should address.
   
   If anyone is interested in working on this, please create sub-tasks under 
this JIRA.
   
   Here is a count by source file as of 19 Dec 2020 of the work "unsafe".
   {code:java}
   ./arrow-pyarrow-integration-testing/src/lib.rs:3
   ./parquet/src/util/hash_util.rs:6
   ./parquet/src/util/bit_packing.rs:34
   ./parquet/src/util/bit_util.rs:1
   ./parquet/src/data_type.rs:12
   ./parquet/src/arrow/record_reader.rs:5
   ./parquet/src/arrow/array_reader.rs:8
   ./arrow/src/buffer.rs:32
   ./arrow/src/compute/kernels/comparison.rs:2
   ./arrow/src/compute/kernels/substring.rs:1
   ./arrow/src/compute/kernels/arithmetic.rs:1
   ./arrow/src/compute/kernels/length.rs:2
   ./arrow/src/compute/util.rs:1
   ./arrow/src/util/bit_chunk_iterator.rs:6
   ./arrow/src/util/bit_util.rs:11
   ./arrow/src/util/integration_util.rs:1
   ./arrow/src/bytes.rs:5
   ./arrow/src/arch/avx512.rs:4
   ./arrow/src/bitmap.rs:1
   ./arrow/src/datatypes.rs:12
   ./arrow/src/zz_memory_check.rs:1
   ./arrow/src/ffi.rs:18
   ./arrow/src/array/transform/mod.rs:1
   ./arrow/src/array/array.rs:1
   ./arrow/src/array/ffi.rs:2
   ./arrow/src/array/array_list.rs:4
   ./arrow/src/array/array_string.rs:2
   ./arrow/src/array/builder.rs:6
   ./arrow/src/array/array_boolean.rs:4
   ./arrow/src/array/raw_pointer.rs:2
   ./arrow/src/array/data.rs:1
   ./arrow/src/array/array_primitive.rs:4
   ./arrow/src/array/array_binary.rs:4
   ./arrow/src/array/array_union.rs:2
   ./arrow/src/memory.rs:10
   ./arrow/src/ipc/gen/File.rs:2
   ./arrow/src/ipc/gen/Message.rs:2
   ./arrow/src/ipc/gen/Schema.rs:2
    {code}
    


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to