Github user zhouxinyu commented on a diff in the pull request:
https://github.com/apache/incubator-rocketmq/pull/145#discussion_r132838496
--- Diff:
broker/src/test/java/org/apache/rocketmq/broker/filter/MessageStoreWithFilterTest.java
---
@@ -201,177 +229,143 @@ public void
Github user zhouxinyu commented on a diff in the pull request:
https://github.com/apache/incubator-rocketmq/pull/145#discussion_r132838396
--- Diff:
broker/src/test/java/org/apache/rocketmq/broker/BrokerControllerTest.java ---
@@ -37,16 +37,14 @@
*/
@Test
Github user zhouxinyu commented on a diff in the pull request:
https://github.com/apache/incubator-rocketmq/pull/145#discussion_r132838549
--- Diff:
client/src/test/java/org/apache/rocketmq/client/consumer/rebalance/AllocateMessageQueueConsitentHashTest.java
---
@@ -95,14 +95,15
Github user Ritabrata-TW commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/134
@zhouxinyu Thanks a lot. Will look forward to working with you again.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user Ritabrata-TW closed the pull request at:
https://github.com/apache/incubator-rocketmq/pull/134
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user zhouxinyu commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/134
@Ritabrata-TW this PR has been merged into `develop` branch and will be
released in next version, you can close it now safely.
---
If your project is set up for it, you can reply to
Github user Jaskey commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/144
Actually, the existing design is that : we have a default name server
service address, which will be specified in our doc(though it is indeed not
demonstrated well for now), the default
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/129
[![Coverage
Status](https://coveralls.io/builds/12817918/badge)](https://coveralls.io/builds/12817918)
Coverage increased (+0.6%) to 39.637% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/145
[![Coverage
Status](https://coveralls.io/builds/12817903/badge)](https://coveralls.io/builds/12817903)
Coverage decreased (-0.06%) to 39.011% when pulling
Github user Jaskey commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/144
@qqeasonchen also, please notice your tittle to make it complete. And link
the jira issue in the desciption
---
If your project is set up for it, you can reply to this email and have
Github user solosky commented on a diff in the pull request:
https://github.com/apache/incubator-rocketmq/pull/133#discussion_r132864458
--- Diff: common/src/main/java/org/apache/rocketmq/common/UtilAll.java ---
@@ -184,15 +184,27 @@ public static String
Github user lizhanhui commented on the issue:
https://github.com/apache/incubator-rocketmq-externals/pull/27
A second issue about alog, there are only header files and binary static
lib file, where is source code for the static lib files?
---
If your project is set up for it, you
Github user lizhanhui commented on the issue:
https://github.com/apache/incubator-rocketmq-externals/pull/27
Trying to create a CMakeLists.txt on top of this PR, but it looks there is
something missing:
```cpp
MetaqJson::Value json
```
I grepped MetaqJson in the code,
Github user lizhanhui commented on the issue:
https://github.com/apache/incubator-rocketmq-externals/pull/27
As it's now part of the Apache RocketMQ project, IMO, rocketmq namespace
should be used instead of metaq.
---
If your project is set up for it, you can reply to this email
14 matches
Mail list logo