Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Partitions with custom reduce are broken #3957

Closed
wants to merge 1 commit into from

Conversation

nickva
Copy link
Contributor

@nickva nickva commented Mar 11, 2022

Don't merge just a draft PR to show the diff

make elixir tests=test/elixir/test/partition_view_test.exs:314

  * test query with custom reduce works (708.2ms)

  1) test query with custom reduce works (ViewPartitionTest)
     test/elixir/test/partition_view_test.exs:314
     Assertion with == failed
     code:  assert resp.status_code == 200
     left:  500
     right: 200
     stacktrace:
       test/elixir/test/partition_view_test.exs:319: (test)

Logs show:

[error] 2022-03-11T20:05:58.969707Z node1@127.0.0.1 <0.985.0> 9bc85227db req_err(2412564908) {{invalid_ejson,{p,<<"bar">>,[<<"field">>,<<"one">>]}},
 [{jiffy,encode,2,
         [{file,"/Users/nvatama/asf-3-main/src/jiffy/src/jiffy.erl"},
          {line,99}]},
  {couch_os_process,writejson,2,[{file,"src/couch_os_process.erl"},{line,97}]},
  {couch_os_process,handle_call,3,

Overview

Testing recommendations

Related Issues or Pull Requests

Checklist

  • Code is written and works correctly
  • Changes are covered by tests
  • Any new configurable parameters are documented in rel/overlay/etc/default.ini
  • A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation

```
make elixir tests=test/elixir/test/partition_view_test.exs:314

  * test query with custom reduce works (708.2ms)

  1) test query with custom reduce works (ViewPartitionTest)
     test/elixir/test/partition_view_test.exs:314
     Assertion with == failed
     code:  assert resp.status_code == 200
     left:  500
     right: 200
     stacktrace:
       test/elixir/test/partition_view_test.exs:319: (test)

```

Logs show:

```
[error] 2022-03-11T20:05:58.969707Z node1@127.0.0.1 <0.985.0> 9bc85227db req_err(2412564908) {{invalid_ejson,{p,<<"bar">>,[<<"field">>,<<"one">>]}},
 [{jiffy,encode,2,
         [{file,"/Users/nvatama/asf-3-main/src/jiffy/src/jiffy.erl"},
          {line,99}]},
  {couch_os_process,writejson,2,[{file,"src/couch_os_process.erl"},{line,97}]},
  {couch_os_process,handle_call,3,
```
@nickva nickva closed this Sep 28, 2023
@nickva nickva deleted the broken-partitioned-custom-reduce branch September 28, 2023 21:02
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.

1 participant