Skip to content

KAFKA-13846: Follow up PR to address review comments#12297

Merged
guozhangwang merged 4 commits intoapache:trunkfrom
vamossagar12:KAFKA-13846-follow-up
Jul 7, 2022
Merged

KAFKA-13846: Follow up PR to address review comments#12297
guozhangwang merged 4 commits intoapache:trunkfrom
vamossagar12:KAFKA-13846-follow-up

Conversation

@vamossagar12
Copy link
Contributor

No description provided.

@vamossagar12
Copy link
Contributor Author

@guozhangwang , Plz review. I addressed the nit comments and updated the documentation by adding a new section for 3.3 as I supposed it wasn't there before.

Copy link
Contributor

@guozhangwang guozhangwang left a comment

Choose a reason for hiding this comment

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

LGTM.

@guozhangwang guozhangwang merged commit 5a1bac2 into apache:trunk Jul 7, 2022
ijuma added a commit to confluentinc/kafka that referenced this pull request Aug 3, 2022
> $ git merge-base apache-github/3.3 apache-github/trunk
> 23c92ce

> $ git show 23c92ce
> commit 23c92ce
> Author: SC <pch838811@gmail.com>
> Date:   Mon Jul 11 11:36:56 2022 +0900
>
>    MINOR: Use String#format for niceMemoryUnits result (apache#12389)
>
>    Reviewers: Luke Chen <showuon@gmail.com>, Divij Vaidya <diviv@amazon.com>

* commit '23c92ce79366e86ca719e5e51c550c27324acd83':
  MINOR: Use String#format for niceMemoryUnits result (apache#12389)
  KAFKA-14055; Txn markers should not be removed by matching records in the offset map (apache#12390)
  KAFKA-13474: Allow reconfiguration of SSL certs for broker to controller connection (apache#12381)
  KAFKA-13996: log.cleaner.io.max.bytes.per.second can be changed dynamically (apache#12296)
  KAFKA-13983: Fail the creation with "/" in resource name in zk ACL (apache#12359)
  KAFKA-12943: update aggregating documentation (apache#12091)
  KAFKA-13846: Follow up PR to address review comments (apache#12297)
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