l1t1 commented on issue #1190: URL: https://github.com/apache/datafusion-python/issues/1190#issuecomment-3120671550
thanks, I find your key modification are: > def merge(self, states: list[pa.Array]) -> None: > # not nice since pyarrow scalars can't be summed yet. This breaks on `None` > self._sum = pa.scalar(self._sum.as_py() + pa.compute.sum(states[0]).as_py()) > > def state(self) -> list[pa.Scalar]: > return [self._sum] I forgot where the original code came from, if it's offical document, it should be modified too. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
