Skip to content

MINOR: Modified Exception handling for KIP-470#7461

Merged
mjsax merged 3 commits intoapache:trunkfrom
jukkakarvanen:KIP-470-exception
Oct 8, 2019
Merged

MINOR: Modified Exception handling for KIP-470#7461
mjsax merged 3 commits intoapache:trunkfrom
jukkakarvanen:KIP-470-exception

Conversation

@jukkakarvanen
Copy link
Contributor

Minor Exception handling changes for KPI-470
IllegalStateException when Provided TestRecord does not have a timestamp and no timestamp overwrite was provided via time parameter.
IllegalStateException when Null keys with readKeyValuesToMap method

Committer Checklist (excluded from commit message)

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

@bbejeck bbejeck added the streams label Oct 7, 2019
Copy link
Member

@bbejeck bbejeck 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 update @jukkakarvanen. LGTM

@bbejeck
Copy link
Member

bbejeck commented Oct 7, 2019

call for second review either of @mjsax or @vvcephei

@mjsax
Copy link
Member

mjsax commented Oct 8, 2019

Java 8 passed.

Java 11 / 2.12: kafka.api.SslAdminClientIntegrationTest.testCreateTopicsResponseMetadataAndConfig

Java 11 / 2.13: kafka.admin.ReassignPartitionsClusterTest.shouldTriggerReassignmentWithZnodePrecedenceOnControllerStartup

Copy link
Member

@mjsax mjsax left a comment

Choose a reason for hiding this comment

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

LGTM. Test failures unrelated. One green build seems sufficient for this PR.

@mjsax mjsax merged commit 7e3f889 into apache:trunk Oct 8, 2019
mjsax pushed a commit that referenced this pull request Oct 8, 2019
ijuma added a commit to ijuma/kafka that referenced this pull request Apr 28, 2020
…t-for-generated-requests

* apache-github/trunk:
  KAFKA-8932; Add tag for CreateTopicsResponse.TopicConfigErrorCode (KIP-525) (apache#7464)
  KAFKA-8944: Fixed KTable compiler warning. (apache#7393)
  KAFKA-8964: Rename tag client-id for thread-level metrics and below (apache#7429)
  MINOR: remove unused imports in Streams system tests (apache#7468)
  KAFKA-7190; Retain producer state until transactionalIdExpiration time passes (apache#7388)
  KAFKA-8983; AdminClient deleteRecords should not fail all partitions unnecessarily (apache#7449)
  MINOR: Modified Exception handling for KIP-470 (apache#7461)
  KAFKA-7245: Deprecate WindowStore#put(key, value) (apache#7105)
  KAFKA-8179: Part 7, cooperative rebalancing in Streams (apache#7386)
  KAFKA-8985; Add flexible version support to inter-broker APIs (apache#7453)
  MINOR: Bump version to 2.5.0-SNAPSHOT (apache#7455)
@mjsax mjsax added the kip Requires or implements a KIP label Jun 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kip Requires or implements a KIP streams

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants