[ 
https://issues.apache.org/jira/browse/STORM-1504?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15124998#comment-15124998
 ] 

ASF GitHub Bot commented on STORM-1504:
---------------------------------------

Github user dossett commented on the pull request:

    https://github.com/apache/storm/pull/1052#issuecomment-177263999
  
    @revans2 @abhishekagarwal87 Addressed your comments and made the serializer 
scheme usable by kryo.  Removed the ConfluentAvroSerializer unit test because I 
can not yet get a usable, test-local schema registry up and running, but it 
does work fine in my dev environment.
    
    @abhishekagarwal87 I could not compile with that shaded package name, but I 
am also not an advanced user of shade, so I could be missing something.  Also, 
I left the GenericSerializer as the default for now.  I would rather have a 
slow default that works and then guide them to better options in documentation, 
but I'm open to counter-arguments.
    
    Still to do, based on additional feedback:
    - refine unit tests one more time
    - add documentation
    
    Thanks!


> AvroGenericRecordBolt will not work in a topology with multiple workers
> -----------------------------------------------------------------------
>
>                 Key: STORM-1504
>                 URL: https://issues.apache.org/jira/browse/STORM-1504
>             Project: Apache Storm
>          Issue Type: Bug
>            Reporter: Aaron Dossett
>            Assignee: Aaron Dossett
>            Priority: Critical
>
> It receives Avro GenericRecord objects, which leads to serialization errors 
> in a multi-worker topology.  Specifically, GenericData.Record will throw a 
> serialization error.
> A serializer for this class should be added and registered.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to