Skip to content

MINOR: Fix doc - FileMessageSet was replaced by FileRecords#4852

Merged
hachikuji merged 1 commit intoapache:trunkfrom
jiminhsieh:fix-doc
Apr 11, 2018
Merged

MINOR: Fix doc - FileMessageSet was replaced by FileRecords#4852
hachikuji merged 1 commit intoapache:trunkfrom
jiminhsieh:fix-doc

Conversation

@jiminhsieh
Copy link
Copy Markdown
Contributor

FileMessageSet no longer existed. It was already replaced by FileRecords.

Committer Checklist (excluded from commit message)

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

@omkreddy
Copy link
Copy Markdown
Contributor

+1

@jiminhsieh jiminhsieh changed the title [MINOR]: Fix doc - FileMessageSet was replaced by FileRecords MINOR: Fix doc - FileMessageSet was replaced by FileRecords Apr 11, 2018
Copy link
Copy Markdown
Contributor

@hachikuji hachikuji left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the patch.

@hachikuji hachikuji merged commit 83a9e04 into apache:trunk Apr 11, 2018
@jiminhsieh jiminhsieh deleted the fix-doc branch April 12, 2018 06:13
koqizhao pushed a commit to koqizhao/kafka that referenced this pull request Apr 12, 2018
…he#4852)

Reviewers: Manikumar Reddy O <manikumar.reddy@gmail.com>, Jason Gustafson <jason@confluent.io>
koqizhao pushed a commit to koqizhao/kafka that referenced this pull request Apr 12, 2018
…he#4852)

Reviewers: Manikumar Reddy O <manikumar.reddy@gmail.com>, Jason Gustafson <jason@confluent.io>
koqizhao pushed a commit to koqizhao/kafka that referenced this pull request Apr 12, 2018
…he#4852)

Reviewers: Manikumar Reddy O <manikumar.reddy@gmail.com>, Jason Gustafson <jason@confluent.io>
ying-zheng pushed a commit to ying-zheng/kafka that referenced this pull request Jul 6, 2018
…he#4852)

Reviewers: Manikumar Reddy O <manikumar.reddy@gmail.com>, Jason Gustafson <jason@confluent.io>
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