Skip to content

MINOR: Fix topology builder debug log message#8005

Merged
mjsax merged 1 commit intoapache:trunkfrom
tedyu:internal-topo-bldr
Jan 28, 2020
Merged

MINOR: Fix topology builder debug log message#8005
mjsax merged 1 commit intoapache:trunkfrom
tedyu:internal-topo-bldr

Conversation

@tedyu
Copy link
Copy Markdown
Contributor

@tedyu tedyu commented Jan 25, 2020

This PR is follow up to the comment from #7969:

#7969 (comment)

Committer Checklist (excluded from commit message)

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

@tedyu
Copy link
Copy Markdown
Contributor Author

tedyu commented Jan 25, 2020

/cc @ableegoldman

@mjsax mjsax added the streams label Jan 25, 2020
@mjsax mjsax changed the title Log size of topics MINOR: Fix topology builder debug log message Jan 25, 2020
@tedyu tedyu force-pushed the internal-topo-bldr branch from 848a5e6 to 7ef4a94 Compare January 25, 2020 02:17
@tedyu tedyu force-pushed the internal-topo-bldr branch from 7ef4a94 to 9031375 Compare January 25, 2020 02:18
@tedyu
Copy link
Copy Markdown
Contributor Author

tedyu commented Jan 25, 2020

@ableegoldman @mjsax
Please take another look.

Copy link
Copy Markdown
Member

@ableegoldman ableegoldman left a comment

Choose a reason for hiding this comment

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

LGTM

@mjsax mjsax merged commit b50e213 into apache:trunk Jan 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants