Skip to content

KAFKA-10246 : AbstractProcessorContext topic() throws NPE#9034

Merged
abbccdda merged 2 commits intoapache:trunkfrom
ashishroy077:trunk
Jul 25, 2020
Merged

KAFKA-10246 : AbstractProcessorContext topic() throws NPE#9034
abbccdda merged 2 commits intoapache:trunkfrom
ashishroy077:trunk

Conversation

@ashishroy077
Copy link
Copy Markdown
Contributor

AbstractProcessorContext topic() throws NullPointerException when modifying a state store within the DSL from a punctuator

…lPointerException when modifying a state store within the DSL from a punctuator
Copy link
Copy Markdown

@abbccdda abbccdda left a comment

Choose a reason for hiding this comment

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

The change LGTM. Could we add a unit test as well?

@ashishroy077
Copy link
Copy Markdown
Contributor Author

Will add a unit test.

@ashishroy077
Copy link
Copy Markdown
Contributor Author

@abbccdda Unit test added.

@abbccdda
Copy link
Copy Markdown

retest this please

@ashishroy077
Copy link
Copy Markdown
Contributor Author

All checks are failing due to some other test case failure. Can anyone look into the logs please?
@abbccdda

@chia7712
Copy link
Copy Markdown
Member

@AshishRoyJava Nice finding and fixing!

AbstractProcessorContext topic() throws NullPointerException when modifying a state store within the DSL from a punctuator

Could you share more details? It seems to me it is weird to see a ProcessorRecordContext carrying null topic.

@abbccdda
Copy link
Copy Markdown

Test failure should be non related.

@abbccdda abbccdda merged commit 483fbd8 into apache:trunk Jul 25, 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)
  ...
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