alamb edited a comment on issue #29: URL: https://github.com/apache/arrow-rs/issues/29#issuecomment-826746621
comment from Paul Kernfeld(paulkernfeld) @ 2019-01-24T00:13:03.622+0000:\n<pre>I'm interested in working on this, although there could be a lot of downstream effects. A good example of a tricky function is arrow::array::PrimitiveArray::value, which appears to be used in a couple dozen places. A few possible strategies are:\r\n # Add in bounds checking so that we don't need to deal with unsafe at all.\r\n # Propagate the unsafes up through the code.\r\n # Maintain a safe and unsafe version of each function that is currently unsafe.\r\n\r\nPersonally I'm a fan of #1 because I think that reducing unsafe code will help developers and users avoid mistakes (I [accidentally wrote|https://github.com/apache/arrow/pull/3448] a nondeterministic unit test earlier this week). However, I'm new to the project so I'm happy to do what others think is best.</pre>\n\ncomment from Paddy Horan(paddyhoran) @ 2019-01-24T01:35:09.674+0000:\n<pre>I was actually thinking we would need #3.\xa0 Taking `value ` as an example I would be in favor of adding bounds checking to `value` and having a `value_unchecked` that does no bounds checking and is unsafe.\r\n\r\nI think that we need to provide the unsafe versions for maximum performance due to Arrow often being described as a development platform rather than a front end API.\xa0 i.e. people will use it as the foundation of other higher level libraries and so developers will want the option to avoid bounds checking for performance reasons.\r\n\r\n\xa0</pre>\n\ncomment from Andrew Lamb(alamb) @ 2021-04-26T10:52:32.742+0000:\n<pre>Migrated to github: https://api.github.com/repos/apache/arrow-rs/issues/28</pre>\n\ncomment from Andrew Lamb(alamb) @ 2021-04-26T10:54:56.406+0000:\n<pre>https://github.com/apache/arrow-rs/issues/28</pre>\n\ncomment from Andrew Lamb(alamb) @ 2021-04-26T11:05:26.331+0000:\n<pre>Testing automated script to migrate issues, trying again</pre>\n\ncomment from Andrew Lamb(alamb) @ 2021-04-26T11:06:54.444+0000:\n<pre>Migr ated to github: https://github.com/apache/arrow-rs/issues/29</pre> -- 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]
