Skip to content

Commit 1e4ac9c

Browse files
authored
fix(transaction): failed to select transaction producer to call local state checker (#246)
* fix(transaction): failed to select transaction producer to call local state checker * fix(transaction): fix the coredump when the transaction producer shutdown.
1 parent d09e7ad commit 1e4ac9c

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

src/MQClientFactory.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
#include "PullRequest.h"
2323
#include "Rebalance.h"
2424
#include "TopicPublishInfo.h"
25-
#include "TransactionMQProducer.h"
25+
#include "TransactionMQProducerImpl.h"
2626

2727
#define MAX_BUFF_SIZE 8192
2828
#define SAFE_BUFF_SIZE 7936 // 8192 - 256 = 7936
@@ -698,7 +698,7 @@ void MQClientFactory::checkTransactionState(const std::string& addr,
698698
if (!group.empty()) {
699699
MQProducer* producer = selectProducer(group);
700700
if (producer != nullptr) {
701-
TransactionMQProducer* transProducer = dynamic_cast<TransactionMQProducer*>(producer);
701+
TransactionMQProducerImpl* transProducer = dynamic_cast<TransactionMQProducerImpl*>(producer);
702702
if (transProducer != nullptr) {
703703
transProducer->checkTransactionState(addr, messageExt, checkRequestHeader.m_tranStateTableOffset,
704704
checkRequestHeader.m_commitLogOffset, checkRequestHeader.m_msgId,

src/extern/CProducer.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -261,10 +261,6 @@ int DestroyProducer(CProducer* pProducer) {
261261
delete defaultMQProducer->innerTransactionProducer;
262262
defaultMQProducer->innerTransactionProducer = NULL;
263263
}
264-
if (defaultMQProducer->listenerInner != NULL) {
265-
delete defaultMQProducer->listenerInner;
266-
defaultMQProducer->listenerInner = NULL;
267-
}
268264
} else {
269265
if (defaultMQProducer->innerProducer != NULL) {
270266
delete defaultMQProducer->innerProducer;

0 commit comments

Comments
 (0)