Skip to content

feat(pouchdb-mapreduce): (#9104) - enable _stats reducer to digest arrays of numbers - #9108

Merged
SourceR85 merged 2 commits into
apache:masterfrom
rednil:master
Aug 6, 2025
Merged

feat(pouchdb-mapreduce): (#9104) - enable _stats reducer to digest arrays of numbers#9108
SourceR85 merged 2 commits into
apache:masterfrom
rednil:master

Conversation

@rednil

@rednil rednil commented Aug 4, 2025

Copy link
Copy Markdown
Contributor

The builtin _sum reducer properly handles numbers AND arrays of numbers, returning an array of sums.
In couchdb, this is also true for the _stats reducer, which returns an array of stats objects.

@SourceR85 SourceR85 changed the title (#9104) - enable _stats reducer to digest arrays of numbers feat(pouchdb-mapreduce): (#9104) - enable _stats reducer to digest arrays of numbers Aug 5, 2025
Comment thread packages/node_modules/pouchdb-mapreduce/src/stats.js
Comment thread tests/mapreduce/test.mapreduce.js
Comment thread tests/mapreduce/test.mapreduce.js
@SourceR85

Copy link
Copy Markdown
Contributor

Well, that doesn't run either...

npm run eslint will show you some required formattings, that must be followed.

Don't worry, sometimes even I forget to run the linter ;)

The builtin _sum reducer properly handles numbers AND arrays of numbers,
returning an array of sums.
In couchdb, this is also true for the _stats reducer, which returns an
array of stats objects.
@rednil

rednil commented Aug 5, 2025

Copy link
Copy Markdown
Contributor Author

bare with me, I will get it ;-)

@rednil
rednil requested a review from SourceR85 August 6, 2025 10:22

@SourceR85 SourceR85 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thank you for your contribution 👍

@SourceR85
SourceR85 merged commit bc4f941 into apache:master Aug 6, 2025
337 of 344 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants