Skip to content
This repository was archived by the owner on Oct 17, 2022. It is now read-only.

Add 2.3.x release notes#353

Merged
wohali merged 1 commit intomasterfrom
2.3.x
Nov 16, 2018
Merged

Add 2.3.x release notes#353
wohali merged 1 commit intomasterfrom
2.3.x

Conversation

@wohali
Copy link
Copy Markdown
Member

@wohali wohali commented Nov 15, 2018

Overview

Time to add the 2.3.x release notes.

Testing recommendations

Read them and make comments!

Checklist

  • Documentation is written and is accurate;
  • make check passes with no errors
  • Update rebar.config.script with the commit hash once this PR is rebased and merged

@wohali wohali requested review from flimzy and nickva November 15, 2018 22:32
@wohali wohali force-pushed the 2.3.x branch 2 times, most recently from 3307c2c to c429e49 Compare November 15, 2018 22:49
@ahayes
Copy link
Copy Markdown
Contributor

ahayes commented Nov 15, 2018

Looks like it will be a good release! I like narwhal.

Copy link
Copy Markdown
Contributor

@nickva nickva left a comment

Choose a reason for hiding this comment

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

Nice work! Thank. Also excited for new features.

Hopefully we can get the rest of the ticket in soon.

@wohali
Copy link
Copy Markdown
Member Author

wohali commented Nov 16, 2018

Want to wait to merge until @janl has had a chance to review, but not sure if he has time. Will merge tomorrow either way.

@janl
Copy link
Copy Markdown
Member

janl commented Nov 16, 2018

+1

@wohali would you be up for rebasing my #328 which already included the 2.3 plumbing?

Comment thread src/whatsnew/2.3.rst Outdated

.. rst-class:: open

* :ghissue:`1602`: To improve security, there has been a major change in the
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think this should be "there have been major changes" ... as it sounds like a list of many changes, not a single change that affects all three (query servers, SSL support, and HTTP global handlers)? Or maybe it was a single change...?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fair enough

Comment thread src/whatsnew/2.3.rst Outdated
Comment thread src/whatsnew/2.3.rst Outdated
Comment thread src/whatsnew/2.3.rst Outdated
Comment thread src/whatsnew/2.3.rst Outdated
Comment thread src/whatsnew/2.3.rst Outdated
of manual config or the cluster setup wizard to bootstrap a cluster. The progress of
the initial seeding of new nodes is exposed at the ``GET /_up`` endpoint.
* Replication supports ipv6-only peers after updating ibrowse dependency.
* :ghissue:`1543`: A CouchDB node can now be restarted via the ``/_node/$node/_restart``
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is mentioned above (lines 101-103). Should these two be merged into a single bullet point?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I can remove this one, the one farther up probably covers it

Comment thread src/whatsnew/2.3.rst Outdated
Comment thread src/whatsnew/2.3.rst
* :ghissue:`1423`: Mango selector matching now occurs at the shard level, reducing the
network traffic within a cluster for a mango query.
* :ghissue:`1423`: Long running operations at the node level could exceed the inter-node
timeout, leading to a fabric timeout error in the logfile and a cancellation of the
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What is a "fabric timeout error"? Perhaps some internal library/daemon/something? Is a less technical term possible, for those not intimately familiar with the internals of CouchDB?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It's an internal library, there's no easier way to explain this. I left "fabric timeout" in there because that exact phrase shows up in the couch logfile if it happens.

Comment thread src/whatsnew/2.3.rst Outdated
Comment thread src/whatsnew/2.3.rst
installations using the ``require_valid_user = true`` setting.
* :ghissue:`1571`: Under very heavy load, it was possible that `rexi_server` could
die in such a way that it's never restarted, leaving a cluster without the ability
to issue RPC calls - effectively rendering the cluster useless.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
to issue RPC calls - effectively rendering the cluster useless.
to issue RPC calls, effectively rendering the cluster useless.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I prefer the dash here. It calls out a subclause that stands out, which it should in this case.

@wohali
Copy link
Copy Markdown
Member Author

wohali commented Nov 16, 2018

@janl I'd rather it stand on its own, since this PR only touches the src/whatsnew directory.... also that PR still needs a minor update before it can land

@wohali wohali merged commit df3934a into master Nov 16, 2018
@wohali wohali deleted the 2.3.x branch November 16, 2018 17:34
@janl
Copy link
Copy Markdown
Member

janl commented Nov 17, 2018

@wohali that's what I meant, my question was whether you're up for taking that over from me. Sorry if I was unclear.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants