Skip to content

MINOR: remove NewTopic#NO_PARTITIONS and NewTopic#NO_REPLICATION_FACT…#9077

Merged
abbccdda merged 1 commit into
apache:trunkfrom
chia7712:KAFKA-REMOVE-NO_PARTITIONS-AND-NO_REPLICATION_FACTOR
Jul 27, 2020
Merged

MINOR: remove NewTopic#NO_PARTITIONS and NewTopic#NO_REPLICATION_FACT…#9077
abbccdda merged 1 commit into
apache:trunkfrom
chia7712:KAFKA-REMOVE-NO_PARTITIONS-AND-NO_REPLICATION_FACTOR

Conversation

@chia7712

Copy link
Copy Markdown
Member

The constants used to represent no_partition and no_replica are important factor in creating topic. Hence, we should have consistent reference to code base.

Committer Checklist (excluded from commit message)

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

…OR as they are duplicate to CreateTopicsRequest#NO_NUM_PARTITIONS and CreateTopicsRequest#NO_REPLICATION_FACTOR
@chia7712 chia7712 force-pushed the KAFKA-REMOVE-NO_PARTITIONS-AND-NO_REPLICATION_FACTOR branch from a164f40 to 4ec482e Compare July 25, 2020 16:19
@abbccdda

Copy link
Copy Markdown

retest this please

@abbccdda

Copy link
Copy Markdown

got 2/3 in previous run

@abbccdda abbccdda left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@chia7712

Copy link
Copy Markdown
Member Author

@abbccdda thanks for reviews!

@chia7712

Copy link
Copy Markdown
Member Author
org.apache.kafka.streams.integration.EosBetaUpgradeIntegrationTest > shouldUpgradeFromEosAlphaToEosBeta[true] FAILED

unrelated failure. retest this please

@abbccdda abbccdda merged commit 659ca8f into apache:trunk Jul 27, 2020
ijuma added a commit to ijuma/kafka that referenced this pull request Nov 17, 2020
…t-for-generated-requests

* apache-github/trunk: (148 commits)
  MINOR: remove NewTopic#NO_PARTITIONS and NewTopic#NO_REPLICATION_FACTOR as they are duplicate to CreateTopicsRequest#NO_NUM_PARTITIONS and CreateTopicsRequest#NO_REPLICATION_FACTOR (apache#9077)
  MINOR: Remove staticmethod tag to be able to use logger of instance (apache#9086)
  MINOR: Adjust 'release.py' script to use shell when using gradlewAll and PGP signing, which were required to build the 2.6.0 RCs (apache#9045)
  MINOR: Update dependencies for Kafka 2.7 (part 1) (apache#9082)
  MINOR: INFO log4j when request re-join (apache#9068)
  MINOR: Recommend Java 11 (apache#9080)
  KAFKA-10306: GlobalThread should fail on InvalidOffsetException (apache#9075)
  KAFKA-10158: Fix flaky testDescribeUnderReplicatedPartitionsWhenReassignmentIsInProgress (apache#9022)
  MINOR: code cleanup for `VOut` inconsistent naming (apache#8907)
  KAFKA-10246 : AbstractProcessorContext topic() throws NPE (apache#9034)
  KAFKA-10305: Print usage when parsing fails for ConsumerPerformance (apache#9071)
  MINOR: removed incorrect deprecation annotations (apache#9061)
  MINOR: speed up release script (apache#9070)
  MINOR: add task ':streams:testAll' (apache#9073)
  KAFKA-10301: Do not clear Partition#remoteReplicasMap during partition assignment updates (apache#9065)
  KAFKA-10268: dynamic config like "--delete-config log.retention.ms" doesn't work (apache#9051)
  KAFKA-10300 fix flaky core/group_mode_transactions_test.py (apache#9059)
  MINOR: Publish metrics package in the javadoc (apache#9036)
  KAFKA-8264: decrease the record size for flaky test
  KAFKA-5876: Add new exception types for Interactive Queries (apache#8200)
  ...
@chia7712 chia7712 deleted the KAFKA-REMOVE-NO_PARTITIONS-AND-NO_REPLICATION_FACTOR branch March 25, 2024 15: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.

2 participants