Skip to content

MINOR: Add explanation for disabling forwarding from value transformers#8771

Merged
mjsax merged 6 commits into
apache:trunkfrom
astubbs:astubbs-forward-disable
Jun 4, 2020
Merged

MINOR: Add explanation for disabling forwarding from value transformers#8771
mjsax merged 6 commits into
apache:trunkfrom
astubbs:astubbs-forward-disable

Conversation

@astubbs

@astubbs astubbs commented Jun 1, 2020

Copy link
Copy Markdown
Contributor

Improve code to be self documenting, in cases where forwarding of messages is disabled due to the type of processor being used,

@astubbs
astubbs force-pushed the astubbs-forward-disable branch from e2c1db5 to be60e9c Compare June 1, 2020 14:07
…formers

Improves code to be self documenting, in cases where forwarding of messages is disabled due to the type of processor being used.
@astubbs
astubbs force-pushed the astubbs-forward-disable branch from be60e9c to c60aa60 Compare June 1, 2020 14:11
@mjsax

mjsax commented Jun 2, 2020

Copy link
Copy Markdown
Member

Retest this please.

@mjsax mjsax added the streams label Jun 2, 2020
@mjsax mjsax changed the title Add explanation for disabling forwarding from value transformers MINOR: Add explanation for disabling forwarding from value transformers Jun 2, 2020

@mjsax mjsax left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@mjsax

mjsax commented Jun 2, 2020

Copy link
Copy Markdown
Member

@astubbs -- build failed with SpotBug error

@astubbs

astubbs commented Jun 2, 2020

Copy link
Copy Markdown
Contributor Author

Retest this please.

@bbejeck

bbejeck commented Jun 2, 2020

Copy link
Copy Markdown
Member

retest this please

@bbejeck

bbejeck commented Jun 2, 2020

Copy link
Copy Markdown
Member

Retest this please.

@mjsax

mjsax commented Jun 2, 2020

Copy link
Copy Markdown
Member

Checkstyle error:

> Task :streams:checkstyleMain
07:47:53 [ant:checkstyle] [ERROR] /home/jenkins/jenkins-slave/workspace/kafka-pr-jdk14-scala2.13/streams/src/main/java/org/apache/kafka/streams/processor/internals/ForwardingDisabledProcessorContext.java:43:33: Name 'explanation' must match pattern '(^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$)|(^log$)'. [ConstantName]

@astubbs

astubbs commented Jun 3, 2020

Copy link
Copy Markdown
Contributor Author

Checkstyle error:

ah dag nammit, sorry guys

@mjsax

mjsax commented Jun 3, 2020

Copy link
Copy Markdown
Member

Retest this please.

@mjsax
mjsax merged commit 542b94b into apache:trunk Jun 4, 2020
mjsax pushed a commit that referenced this pull request Jun 4, 2020
…rs (#8771)

Reviewers: Matthias J. Sax <matthias@confluent.io>, Bill Bejeck <bill@confluent.io>
@mjsax

mjsax commented Jun 4, 2020

Copy link
Copy Markdown
Member

Thanks for the PR @astubbs!

Merged to trunk and cherry-picked to 2.6.

Kvicii pushed a commit to Kvicii/kafka that referenced this pull request Jun 6, 2020
* 'trunk' of github.com:apache/kafka: (46 commits)
  MINOR: improve code encapsulation between StreamThread and TaskManager (apache#8819)
  Fixing KAFKA-10094 (apache#8797)
  KAFKA-9851: Revoking Connect tasks due to connectivity issues should also clear the running assignment (apache#8804)
  KAFKA-9840; Skip End Offset validation when the leader epoch is not reliable (apache#8486)
  HOT_FIX: Update javadoc since imports added (apache#8817)
  KAFKA-8011: Fix flaky RegexSourceIntegrationTest (apache#8799)
  KAFKA-9570: Define SSL configs in all worker config classes, not just distributed (apache#8135)
  KAFKA-10111: Make SinkTaskContext.errantRecordReporter() a default method (apache#8814)
  KAFKA-10110: Corrected potential NPE when null label value added to KafkaMetricsContext (apache#8811)
  MINOR: Change the order that Connect calls `config()` and `validate()` to avoid validating if the required ConfigDef is null (apache#8810)
  MINOR: fix backwards incompatibility in JmxReporter introduced by KIP-606
  MINOR: Fix javadoc warnings (apache#8809)
  KAFKA-9441: Improve Kafka Streams task management (apache#8776)
  fix the broken links of streams javadoc (apache#8789)
  KAFKA-10040; Make computing the PreferredReplicaImbalanceCount metric more efficient (apache#8724)
  KAFKA-10066: TestOutputTopic should pass record headers into deserializers (apache#8759)
  MINOR: Add explanation for disabling forwarding from value transformers (apache#8771)
  KAFKA-10033: Throw UnknownTopicOrPartitionException if altering configs of non-existing topic
  KAFKA-9434: automated protocol for alterReplicaLogDirs (apache#8311)
  KAFKA-9313: Set `use_all_dns_ips` as the new default for `client.dns.lookup` (KIP-602) (apache#8644)
  ...
@astubbs
astubbs deleted the astubbs-forward-disable branch June 8, 2020 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants