-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50318/
-----------------------------------------------------------

Review request for samza, Navina Ramesh and Yi Pan (Data Infrastructure).


Repository: samza


Description
-------

After the thread-safe kafka producer change, all producer need to register the 
source before publishing to it. The registering of KafkaCheckpointMigration 
happens in the wrong place. Move it after the producer is created.

Also find an extra logging introduced by the change. Remove it during this fix.


Diffs
-----

  samza-core/src/main/scala/org/apache/samza/checkpoint/OffsetManager.scala 
7245902c69c751a4e8853745de46adf5553d45f5 
  
samza-kafka/src/main/scala/org/apache/samza/migration/KafkaCheckpointMigration.scala
 5d2641ab1e54d49f7b983bc526762cfb50f2911b 

Diff: https://reviews.apache.org/r/50318/diff/


Testing
-------

Tested by locally deployed jobs.

Passed gradle tests.


Thanks,

Xinyu Liu

Reply via email to