Browse Source

Fix bug where we were waiting for a response when we shouldn't have been

Evan Huus 12 years ago
parent
commit
e0f622e045
1 changed files with 6 additions and 3 deletions
  1. 6 3
      protocol/broker.go

+ 6 - 3
protocol/broker.go

@@ -128,12 +128,15 @@ func (b *Broker) GetAvailableOffsets(clientID string, request *OffsetRequest) (*
 
 
 func (b *Broker) Produce(clientID string, request *ProduceRequest) (*ProduceResponse, error) {
 func (b *Broker) Produce(clientID string, request *ProduceRequest) (*ProduceResponse, error) {
 	var response *ProduceResponse
 	var response *ProduceResponse
-	if request.ResponseCondition != NO_RESPONSE {
+	var err error
+
+	if request.ResponseCondition == NO_RESPONSE {
+		err = b.sendAndReceive(clientID, request, nil)
+	} else {
 		response = new(ProduceResponse)
 		response = new(ProduceResponse)
+		err = b.sendAndReceive(clientID, request, response)
 	}
 	}
 
 
-	err := b.sendAndReceive(clientID, request, response)
-
 	if err != nil {
 	if err != nil {
 		return nil, err
 		return nil, err
 	}
 	}