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

    https://github.com/apache/storm/pull/2588#discussion_r173635613
  
    --- Diff: 
external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/format/DefaultEventDataFormat.java
 ---
    @@ -24,24 +24,24 @@
      * into a delimited string.
      */
     public class DefaultEventDataFormat implements IEventDataFormat {
    -  private static final long serialVersionUID = 1L;
    -  private String delimiter = ",";
    -  
    -  public DefaultEventDataFormat withFieldDelimiter(String delimiter) {
    -    this.delimiter = delimiter;
    -    return this;
    -  }
    +    private static final long serialVersionUID = 1L;
    +    private String delimiter = ",";
     
    -  @Override
    -  public byte[] serialize(Tuple tuple) {
    -    StringBuilder sb = new StringBuilder();
    -    for(Object obj : tuple.getValues()) {
    -      if(sb.length() != 0) {
    -        sb.append(delimiter);
    -      }
    -      sb.append(obj.toString());
    +    public DefaultEventDataFormat withFieldDelimiter(String delimiter) {
    +        this.delimiter = delimiter;
    +        return this;
    +    }
    +
    +    @Override
    +    public byte[] serialize(Tuple tuple) {
    +        StringBuilder sb = new StringBuilder();
    +        for (Object obj : tuple.getValues()) {
    +            if (sb.length() != 0) {
    +                sb.append(delimiter);
    +            }
    +            sb.append(obj.toString());
    +        }
    +        return sb.toString().getBytes();
    --- End diff --
    
    Get bytes without charset


---

Reply via email to