Github user dossett commented on a diff in the pull request:

    https://github.com/apache/storm/pull/1052#discussion_r51609059
  
    --- Diff: external/storm-hdfs/README.md ---
    @@ -315,6 +314,18 @@ An `org.apache.avro.Schema` object cannot be directly 
provided since it does not
     The AvroGenericRecordBolt expects to receive tuples containing an Avro 
GenericRecord that conforms to the provided
     schema.
     
    +To use this bolt you **must** register the appropriate Kryo serializers 
with your topology configuration.  A convenience
    +method is provided for this:
    +
    +```AvroGenericRecordBolt.addAvroKryoSerializations(conf);```
    +
    +By default Storm will use the ```GenericAvroSerializer``` to handle 
serialization.  This will work, but there are much 
    +faster options available if you can pre-define the schemas you will be 
using or utilize an external schema registry. An
    +implementation using the Confluent Schema Registry is provided, but others 
can be implemented and provided to Storm.
    +Please see the javadoc for classes in org.apache.storm.hdfs.avro for 
information about using the built-in options or
    +creating your own.
    +
    --- End diff --
    
    It looked good in IntelliJ, thanks for checking it here. Should be fixed 
now.


---
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