Jun Rao created KAFKA-1302:
------------------------------

             Summary: cleanup logging in new producer
                 Key: KAFKA-1302
                 URL: https://issues.apache.org/jira/browse/KAFKA-1302
             Project: Kafka
          Issue Type: Bug
          Components: core
    Affects Versions: 0.8.2
            Reporter: Jun Rao
            Assignee: Jun Rao


1. When we hit an error in producer, we call the callback with a null 
RecordMetadata. If the callback tries to get the topic/partition, it will hit a 
NullPointerException. It's probably better to create a RecordMetadata with -1 
as the offset.
2. When printing out a Struct, we don't print out the content wrapped in an 
array properly. So, we will see sth like the following.
[2014-03-09 11:56:24,364] INFO Created 1 requests: 
[InFlightRequest(expectResponse=true, 
batches={test-0=RecordBatch(topicPartition=test-0, recordCount=1)}, 
request=RequestSend(header={api_key=0,api_version=0,correlation_id=1,client_id=perf-test},
 body={acks=-1,timeout=3000,topic_data=[Ljava.lang.Object;@700a4488}))] 
(org.apache.kafka.clients.producer.internals.Sender)
3. Need to override the toString() in ProduceResponse.
4. Sender.run(now): It would be good to log metadata request too.
5. Sender.handleDisconnects(): It would be useful to log the correlation id of 
cancelled inflight requests




--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to