drin commented on PR #13583: URL: https://github.com/apache/arrow/pull/13583#issuecomment-1182562069
I just realized that this introduces (or maybe just exposes) a bug when calling this function on scalar inputs. If the input is a scalar, `non_nulls` is incremented without changing state. To address this "correctly," the code path for scalar inputs should also update the state by using `GetOrInsert` as is done for the code path for vector inputs. I am working on figuring this out, given that the compiler has type conversion issues when attempting to just call `GetOrInsert` with a scalar reference. -- 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]
