Github user jvwing commented on the issue:

    https://github.com/apache/nifi/pull/2409
  
    @SunSatION , thanks for submitting this PR, it looks like good stuff.
    
    May I ask how you tested the changes?  For PutKinesisFirehose, I was able 
to run the integration test suite successfully, and I was able to manually 
build and run a simple flow sending data to Kinesis Firehose.  For 
PutKinesisStream, the manual flow building worked, but I get exceptions running 
the integration test suite (ITPutKinesisStream.testIntegrationSuccess) as it 
calls `putRecords()`:
    
    ```
    com.amazonaws.SdkClientException: Unable to marshall request to JSON: 
    
com.fasterxml.jackson.dataformat.cbor.CBORGenerator.getOutputContext()Lcom/fasterxml/jackson/core/json/JsonWriteContext;
    ```
    
    Did you try the integration tests, and did they work for you?  I don't 
believe your change introduced this issue.  A quick check of the master branch 
suggests it was already there, but it complicated testing.


---

Reply via email to