/**
 * Licensed to the Apache Software Foundation (ASF) under one
 * or more contributor license agreements.  See the NOTICE file
 * distributed with this work for additional information
 * regarding copyright ownership.  The ASF licenses this file
 * to you under the Apache License, Version 2.0 (the
 * "License"); you may not use this file except in compliance
 * with the License.  You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package storm.kafka.trident;

import backtype.storm.task.OutputCollector;
import backtype.storm.topology.FailedException;
import org.apache.commons.lang.Validate;
import org.apache.kafka.clients.producer.Callback;
import org.apache.kafka.clients.producer.KafkaProducer;
import org.apache.kafka.clients.producer.ProducerRecord;
import org.apache.kafka.clients.producer.RecordMetadata;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import storm.kafka.trident.mapper.TridentTupleToKafkaMapper;
import storm.kafka.trident.selector.KafkaTopicSelector;
import storm.trident.operation.TridentCollector;
import storm.trident.state.State;
import storm.trident.tuple.TridentTuple;

import java.util.List;
import java.util.Map;
import java.util.Properties;

public class TridentKafkaState implements State {
    private static final Logger LOG = LoggerFactory.getLogger(TridentKafkaState.class);

    public static final String KAFKA_BROKER_PROPERTIES = "kafka.broker.properties";

    private KafkaProducer producer;
    private OutputCollector collector;

    private TridentTupleToKafkaMapper mapper;
    private KafkaTopicSelector topicSelector;

    public TridentKafkaState withTridentTupleToKafkaMapper(TridentTupleToKafkaMapper mapper) {
        this.mapper = mapper;
        return this;
    }

    public TridentKafkaState withKafkaTopicSelector(KafkaTopicSelector selector) {
        this.topicSelector = selector;
        return this;
    }

    @Override
    public void beginCommit(Long txid) {
        LOG.debug("beginCommit is Noop.");
    }

    @Override
    public void commit(Long txid) {
        LOG.debug("commit is Noop.");
    }

    public void prepare(Map stormConf) {
        Validate.notNull(mapper, "mapper can not be null");
        Validate.notNull(topicSelector, "topicSelector can not be null");
        Map configMap = (Map) stormConf.get(KAFKA_BROKER_PROPERTIES);
        Properties properties = new Properties();
        properties.putAll(configMap);
        producer = new KafkaProducer(properties);
    }

    public void updateState(List<TridentTuple> tuples, TridentCollector collector) {
        for (final TridentTuple tuple : tuples) {
            final String topic = topicSelector.getTopic(tuple);
            if(topic != null) {
                producer.send(new ProducerRecord(topic, mapper.getKeyFromTuple(tuple),
                        mapper.getMessageFromTuple(tuple)),new Callback() {
                            @Override
                            public void onCompletion(RecordMetadata metadata, Exception ex) {
                                if(ex != null){
                                    String errorMsg = "Could not send message with key = "
                                    + mapper.getKeyFromTuple(tuple) + " to topic = " + topic;
                                    LOG.warn(errorMsg, ex);
                                    throw new FailedException(errorMsg, ex);
                                }
                            }
                        });
            } else {
                LOG.warn("skipping key = " + mapper.getKeyFromTuple(tuple) + ", topic selector returned null.");
            }
        }
    }
}

@@ -963,59 +963,8 @@
 te;%0A
-import org.apache.kafka.clients.producer.Callback;%0A
 impo
@@ -1076,65 +1076,8 @@
 rd;%0A
-import org.apache.kafka.clients.producer.RecordMetadata;%0A
 impo
@@ -2829,19 +2829,42 @@
     
-for (final 
+String topic = null;%0A        for (
 Trid
@@ -2902,28 +2902,37 @@
         
-final String
+try %7B%0A               
  topic =
@@ -2963,16 +2963,17 @@
 tuple);%0A
+%0A
         
@@ -2976,16 +2976,20 @@
         
+    
 if(topic
@@ -3008,32 +3008,36 @@
                 
+ 
+   
 producer.send(ne
@@ -3115,16 +3115,20 @@
         
+    
 mapper.g
@@ -3157,157 +3157,158 @@
 le))
-,new Callback() %7B%0A                            @Override%0A                            public void onCompletion(RecordMetadata metadata, Exception ex) %7B
+).get();%0A                %7D else %7B%0A                    LOG.warn(%22skipping key = %22 + mapper.getKeyFromTuple(tuple) + %22, topic selector returned null.%22);
 %0A   
@@ -3312,32 +3312,34 @@
                 
+%7D%0A
                 
@@ -3338,233 +3338,186 @@
     
-    if(ex != null)%7B%0A                                    String errorMsg = %22Could not send message with key = %22%0A                                    + mapper.getKeyFromTuple(tuple) + %22 to topic = %22 + topic;%0A                    
+%7D catch (Exception ex) %7B%0A                String errorMsg = %22Could not send message with key = %22 + mapper.getKeyFromTuple(tuple)%0A                        + %22 to topic = %22 + topic;%0A
     
@@ -3552,36 +3552,16 @@
 g, ex);%0A
-                    
         
@@ -3613,234 +3613,8 @@
 x);%0A
-                                %7D%0A                            %7D%0A                        %7D);%0A            %7D else %7B%0A                LOG.warn(%22skipping key = %22 + mapper.getKeyFromTuple(tuple) + %22, topic selector returned null.%22);%0A
     
@@ -3615,32 +3615,32 @@
 ;%0A            %7D%0A
-
         %7D%0A    %7D%0A
@@ -3639,10 +3639,16 @@
 %7D%0A    %7D%0A
+    %7D%0A
 %7D%0A

use sync send method