Skip to content

backport: enable the ADR 075 event log by default in new configs in rpc (#8572)#9856

Merged
thanethomson merged 1 commit intotendermint:feature/adr075-backportfrom
mmsqe:feature/adr075-backport-default-event-log
Dec 8, 2022
Merged

backport: enable the ADR 075 event log by default in new configs in rpc (#8572)#9856
thanethomson merged 1 commit intotendermint:feature/adr075-backportfrom
mmsqe:feature/adr075-backport-default-event-log

Conversation

@mmsqe
Copy link

@mmsqe mmsqe commented Dec 8, 2022

This is the last backport for adr 075, for the full description, see #8572


PR checklist

  • Tests written/updated, or no tests needed
  • CHANGELOG_PENDING.md updated, or no changelog entry needed
  • Updated relevant documentation (docs/) and code comments, or no
    documentation updates needed

@mmsqe mmsqe marked this pull request as ready for review December 8, 2022 01:15
@mmsqe mmsqe requested a review from ebuchman as a code owner December 8, 2022 01:15
@mmsqe mmsqe requested a review from a team December 8, 2022 01:15
@mmsqe mmsqe force-pushed the feature/adr075-backport-default-event-log branch from a38c2d3 to e1a14b0 Compare December 8, 2022 01:33
…nt#8572)

Since we are deprecating the stream-based event subscription in v0.36, we
should ensure that new nodes enable the replacement by default.  For now, just
set a baseline 30-second window.
@mmsqe mmsqe force-pushed the feature/adr075-backport-default-event-log branch from e1a14b0 to ccb3b18 Compare December 8, 2022 02:54
@thanethomson thanethomson merged commit 5eac8d7 into tendermint:feature/adr075-backport Dec 8, 2022
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