Skip to content

MINOR: Caching layer should forward record timestamp (#5423)#5426

Merged
guozhangwang merged 1 commit into
apache:1.1from
mjsax:minor-window-timestamp-11
Jul 26, 2018
Merged

MINOR: Caching layer should forward record timestamp (#5423)#5426
guozhangwang merged 1 commit into
apache:1.1from
mjsax:minor-window-timestamp-11

Conversation

@mjsax

@mjsax mjsax commented Jul 26, 2018

Copy link
Copy Markdown
Member

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.

Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

Reviewer: Guozhang Wang <guozhang@confluent.io>
@mjsax

mjsax commented Jul 26, 2018

Copy link
Copy Markdown
Member Author

Call for review @guozhangwang @bbejeck

Was not a clean cherry-pick from #5423 so doing a new PR for 1.1

@guozhangwang
guozhangwang merged commit 3b6e0f3 into apache:1.1 Jul 26, 2018
mjsax added a commit that referenced this pull request Aug 1, 2018
Reviewer: Guozhang Wang <guozhang@confluent.io>
mjsax added a commit that referenced this pull request Aug 1, 2018
Reviewer: Guozhang Wang <guozhang@confluent.io>
mjsax added a commit that referenced this pull request Aug 1, 2018
Reviewer: Guozhang Wang <guozhang@confluent.io>
mjsax added a commit that referenced this pull request Aug 1, 2018
Reviewer: Guozhang Wang <guozhang@confluent.io>
@mjsax

mjsax commented Aug 1, 2018

Copy link
Copy Markdown
Member Author

Cherry-picked to 1.0, 0.11.0, 0.10.2, and 0.10.1.

@mjsax
mjsax deleted the minor-window-timestamp-11 branch August 1, 2018 22:22
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.

3 participants