Skip to content

MINOR: Log partition info when creating new request batch in controller#6145

Merged
ijuma merged 1 commit intoapache:trunkfrom
jaceklaskowski:trunk
Jan 19, 2019
Merged

MINOR: Log partition info when creating new request batch in controller#6145
ijuma merged 1 commit intoapache:trunkfrom
jaceklaskowski:trunk

Conversation

@jaceklaskowski
Copy link
Copy Markdown
Contributor

@jaceklaskowski jaceklaskowski commented Jan 15, 2019

Due to the missing $, the name was being logged instead of the value.

@jaceklaskowski
Copy link
Copy Markdown
Contributor Author

Hey committers, any chance to review such a small PR? Thanks.

Copy link
Copy Markdown
Member

@ijuma ijuma left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, LGTM.

@ijuma ijuma changed the title MINOR: Display partition info in controller logs MINOR: Log partition info when creating new request batch in controller Jan 19, 2019
@ijuma ijuma merged commit 59f2252 into apache:trunk Jan 19, 2019
@ijuma
Copy link
Copy Markdown
Member

ijuma commented Jan 19, 2019

Merged to trunk and cherry-picked to the 2.1 branch.

ijuma pushed a commit that referenced this pull request Jan 19, 2019
…er (#6145)

Due to the missing `$`, the name was being logged instead of the value.

Reviewers: Ismael Juma <ismael@juma.me.uk>
abbccdda pushed a commit to abbccdda/kafka that referenced this pull request Jan 24, 2019
…er (apache#6145)

Due to the missing `$`, the name was being logged instead of the value.

Reviewers: Ismael Juma <ismael@juma.me.uk>
jarekr pushed a commit to confluentinc/kafka that referenced this pull request Apr 18, 2019
* ak/trunk:
  MINOR: fix race condition in KafkaStreamsTest (apache#6185)
  KAFKA-4850: Enable bloomfilters (apache#6012)
  MINOR: ducker-ak: add down -f, avoid using a terminal in ducker test
  KAFKA-5117: Stop resolving externalized configs in Connect REST API
  MINOR: Cleanup handling of mixed transactional/idempotent records (apache#6172)
  KAFKA-7844: Use regular subproject for generator to fix *All targets (apache#6182)
  Fix Documentation for cleanup.policy is out of date (apache#6181)
  MINOR: increase timeouts for KafkaStreamsTest (apache#6178)
  MINOR: Rejoin split ssl principal mapping rules (apache#6099)
  MINOR: Handle case where connector status endpoints returns 404 (apache#6176)
  MINOR: Remove unused imports, exceptions, and values (apache#6117)
  KAFKA-3522: Add internal RecordConverter interface (apache#6150)
  Fix Javadoc of KafkaConsumer (apache#6155)
  KAFKA-6455: Extend CacheFlushListener to forward timestamp (apache#6147)
  MINOR: Log partition info when creating new request batch in controller (apache#6145)
  KAFKA-7652: Part I; Fix SessionStore's findSession(single-key) (apache#6134)
  MINOR: Remove the InvalidTopicException handling in InternalTopicManager (apache#6167)
  [KAFKA-7024] Rocksdb state directory should be created before opening the DB (apache#6138)
  MINOR:: Fix typos (apache#6079)
pengxiaolong pushed a commit to pengxiaolong/kafka that referenced this pull request Jun 14, 2019
…er (apache#6145)

Due to the missing `$`, the name was being logged instead of the value.

Reviewers: Ismael Juma <ismael@juma.me.uk>
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