Github user Parth-Brahmbhatt commented on a diff in the pull request:

    https://github.com/apache/storm/pull/347#discussion_r21839632
  
    --- Diff: 
external/storm-hdfs/src/main/java/org/apache/storm/hdfs/bolt/SequenceFileBolt.java
 ---
    @@ -108,7 +109,11 @@ public void execute(Tuple tuple) {
             try {
                 long offset;
                 synchronized (this.writeLock) {
    -                this.writer.append(this.format.key(tuple), 
this.format.value(tuple));
    +                if (this.format instanceof SerializableSequenceFormat) {
    --- End diff --
    
    The new interface and type checking is only necessary for backwards 
compatibility, you could change the return type of original SequenceFormat 
interface to Object and no other code change will be necessary . I would add 
that as a comment here so when we decide to do a major version bump we could 
get rid of this extra code and the extra interface.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to