提交 3ccd6f97 编写于 作者: D duheng 提交者: Zhanhui Li

Fix issues brought about while merging pull requests

上级 17648c67
...@@ -24,7 +24,7 @@ import org.apache.commons.lang3.StringUtils; ...@@ -24,7 +24,7 @@ import org.apache.commons.lang3.StringUtils;
import org.apache.rocketmq.client.consumer.AllocateMessageQueueStrategy; import org.apache.rocketmq.client.consumer.AllocateMessageQueueStrategy;
import org.apache.rocketmq.client.log.ClientLogger; import org.apache.rocketmq.client.log.ClientLogger;
import org.apache.rocketmq.common.message.MessageQueue; import org.apache.rocketmq.common.message.MessageQueue;
import org.slf4j.Logger; import org.apache.rocketmq.logging.InternalLogger;
/** /**
* An allocate strategy proxy for based on machine room nearside priority. An actual allocate strategy can be * An allocate strategy proxy for based on machine room nearside priority. An actual allocate strategy can be
...@@ -35,7 +35,7 @@ import org.slf4j.Logger; ...@@ -35,7 +35,7 @@ import org.slf4j.Logger;
* no alive consumer to monopolize them. * no alive consumer to monopolize them.
*/ */
public class AllocateMachineRoomNearby implements AllocateMessageQueueStrategy { public class AllocateMachineRoomNearby implements AllocateMessageQueueStrategy {
private final Logger log = ClientLogger.getLog(); private final InternalLogger log = ClientLogger.getLog();
private final AllocateMessageQueueStrategy allocateMessageQueueStrategy;//actual allocate strategy private final AllocateMessageQueueStrategy allocateMessageQueueStrategy;//actual allocate strategy
private final MachineRoomResolver machineRoomResolver; private final MachineRoomResolver machineRoomResolver;
......
...@@ -36,7 +36,7 @@ public class NettyRemotingAbstractTest { ...@@ -36,7 +36,7 @@ public class NettyRemotingAbstractTest {
@Test @Test
public void testProcessResponseCommand() throws InterruptedException { public void testProcessResponseCommand() throws InterruptedException {
final Semaphore semaphore = new Semaphore(0); final Semaphore semaphore = new Semaphore(0);
ResponseFuture responseFuture = new ResponseFuture(1, 3000, new InvokeCallback() { ResponseFuture responseFuture = new ResponseFuture(null,1, 3000, new InvokeCallback() {
@Override @Override
public void operationComplete(final ResponseFuture responseFuture) { public void operationComplete(final ResponseFuture responseFuture) {
assertThat(semaphore.availablePermits()).isEqualTo(0); assertThat(semaphore.availablePermits()).isEqualTo(0);
...@@ -57,7 +57,7 @@ public class NettyRemotingAbstractTest { ...@@ -57,7 +57,7 @@ public class NettyRemotingAbstractTest {
@Test @Test
public void testProcessResponseCommand_NullCallBack() throws InterruptedException { public void testProcessResponseCommand_NullCallBack() throws InterruptedException {
final Semaphore semaphore = new Semaphore(0); final Semaphore semaphore = new Semaphore(0);
ResponseFuture responseFuture = new ResponseFuture(1, 3000, null, ResponseFuture responseFuture = new ResponseFuture(null,1, 3000, null,
new SemaphoreReleaseOnlyOnce(semaphore)); new SemaphoreReleaseOnlyOnce(semaphore));
remotingAbstract.responseTable.putIfAbsent(1, responseFuture); remotingAbstract.responseTable.putIfAbsent(1, responseFuture);
...@@ -72,7 +72,7 @@ public class NettyRemotingAbstractTest { ...@@ -72,7 +72,7 @@ public class NettyRemotingAbstractTest {
@Test @Test
public void testProcessResponseCommand_RunCallBackInCurrentThread() throws InterruptedException { public void testProcessResponseCommand_RunCallBackInCurrentThread() throws InterruptedException {
final Semaphore semaphore = new Semaphore(0); final Semaphore semaphore = new Semaphore(0);
ResponseFuture responseFuture = new ResponseFuture(1, 3000, new InvokeCallback() { ResponseFuture responseFuture = new ResponseFuture(null,1, 3000, new InvokeCallback() {
@Override @Override
public void operationComplete(final ResponseFuture responseFuture) { public void operationComplete(final ResponseFuture responseFuture) {
assertThat(semaphore.availablePermits()).isEqualTo(0); assertThat(semaphore.availablePermits()).isEqualTo(0);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册