andishgar commented on issue #49179: URL: https://github.com/apache/arrow/issues/49179#issuecomment-3901534143
@kou, I have several notes: 1-I checked the PR and its subsequent issue; however, my proposal requires a more radical change. To clarify, I suggest changing the C type of HalfFloatType from uint16_t to arrow::util::Float16 (see [here](https://github.com/apache/arrow/blob/134638d2d3a5b358549d6650a0021b2220607556/cpp/src/arrow/type.h#L707)). The existing issue focuses only on scalar types. Given this, should I open a new issue? 2-From the CI logs of #47519, I found that this change requires modifications in Ruby, R, and Python. Could I ask for help applying the changes in these languages? -- 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]
