GitHub user XuMingmin opened a pull request:

    https://github.com/apache/beam/pull/2044

    [BEAM-1407] Support multiple Kafka client in KakaIO

    PR for *[Solution 3. Rewrite Consumer method call at runtime].* SpEL is 
used to hide the difference of Kafka client 0.9/0.10.
    
    Additional tests are listed below:
    * Kafka client 0.9.0.1
     1. Kafka9 -> Kafka9   [PASS]  *(means READ from Kafka9, then WRITE to 
Kafka9)*
     2. Kafka9 -> Kafka10  [PASS]
     3. Kafka10 -> Kafka9  [PASS]
     4. Kafka10 -> Kafka10 [PASS]
    * Kafka client 0.10.0, with/without external authentication implementation
     1. Kafka9 -> Kafka9   [FAIL]
     2. Kafka9 -> Kafka10  [FAIL]
     3. Kafka10 -> Kafka9  [FAIL]
     4. Kafka10 -> Kafka10 [PASS]
    
    *As Kakfa client 0.10 is not compatible with Kafka cluster 0.9, the 3 items 
are expected to fail.*   
    


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/XuMingmin/beam master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/beam/pull/2044.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2044
    
----
commit 7f6cc88604eae97b444b26adabf8ba4c0eb0f8fe
Author: XuMingmin <[email protected]>
Date:   2017-02-09T21:36:39Z

    Merge pull request #1 from apache/master
    
    sync up code from apache/beam

commit e15b1cfb17058a6ee008ae54b980921b890ff36c
Author: mingmxu <[email protected]>
Date:   2017-02-18T07:11:19Z

    add SpEL to hide kafka client difference of 0.9/0.19

commit 57480b361b7f927bb8b6f3398070709b99eaa06c
Author: XuMingmin <[email protected]>
Date:   2017-02-18T07:19:20Z

    Merge pull request #2 from apache/master
    
    pull latest code

commit fb3a8f137c9ce9ad42ce01973aa8a8776e15c082
Author: mingmxu <[email protected]>
Date:   2017-02-18T07:32:32Z

    document the change

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to