You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/08/20 01:50:57 UTC
[GitHub] [arrow] cyb70289 opened a new pull request #10967: ARROW-13520: [C++] Simplify TDigest.Merge API
cyb70289 opened a new pull request #10967:
URL: https://github.com/apache/arrow/pull/10967
Current code performs k-way merge on multiple tdigests. Though looks
fancy, it's not very useful in practice. More importantly, it brings
big headache on how to maintain the input tdigest vector decently.
This patch simplifies the API to merge only one tdigest once.
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] github-actions[bot] commented on pull request #10967: ARROW-13682: [C++] Add TDigest API to merge one TDigest
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #10967:
URL: https://github.com/apache/arrow/pull/10967#issuecomment-902663550
https://issues.apache.org/jira/browse/ARROW-13682
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] github-actions[bot] commented on pull request #10967: ARROW-13520: [C++] Simplify TDigest.Merge API
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #10967:
URL: https://github.com/apache/arrow/pull/10967#issuecomment-902369055
https://issues.apache.org/jira/browse/ARROW-13520
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] cyb70289 commented on pull request #10967: ARROW-13682: [C++] Add TDigest API to merge one TDigest
Posted by GitBox <gi...@apache.org>.
cyb70289 commented on pull request #10967:
URL: https://github.com/apache/arrow/pull/10967#issuecomment-902725644
Thanks for creating a jira issue.
I created one for this pr. But obviously I messed it up.
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] lidavidm closed pull request #10967: ARROW-13682: [C++] Add TDigest API to merge one TDigest
Posted by GitBox <gi...@apache.org>.
lidavidm closed pull request #10967:
URL: https://github.com/apache/arrow/pull/10967
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org