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)