Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/85
[![Coverage
Status](https://coveralls.io/builds/10821765/badge)](https://coveralls.io/builds/10821765)
Coverage decreased (-0.08%) to 31.791% when pulling
Github user lizhanhui commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/85
> So do you agree this is indeed an issue that needed to be solved? If
yes, I will try doing that.
Changing specific dependency to a neutral logging framework is good to have
Github user Jaskey commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/85
@lizhanhui
So do you agree this is indeed an issue that needed to be solved? If yes,
I will try doing that.
But actually it is a little different from the clientLogger
Github user lizhanhui commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/79
Merged
---
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,
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/85
[![Coverage
Status](https://coveralls.io/builds/10807112/badge)](https://coveralls.io/builds/10807112)
Coverage increased (+0.04%) to 31.662% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/85
[![Coverage
Status](https://coveralls.io/builds/10806993/badge)](https://coveralls.io/builds/10806993)
Coverage increased (+0.03%) to 31.644% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/85
[![Coverage
Status](https://coveralls.io/builds/10806993/badge)](https://coveralls.io/builds/10806993)
Coverage increased (+0.03%) to 31.644% when pulling
GitHub user hustfxj opened a pull request:
https://github.com/apache/incubator-rocketmq-externals/pull/5
[ROCKETMQ-81] import rocketmq plugin for spark streaming
This plugin is for spark streaming based on pull mode. And you can see
READ.ME for more details.
You can merge this
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/79
[![Coverage
Status](https://coveralls.io/builds/10803343/badge)](https://coveralls.io/builds/10803343)
Coverage increased (+1.0%) to 31.798% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/79
[![Coverage
Status](https://coveralls.io/builds/10803343/badge)](https://coveralls.io/builds/10803343)
Coverage increased (+1.0%) to 31.798% when pulling
Github user coveralls commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/79
[![Coverage
Status](https://coveralls.io/builds/10803343/badge)](https://coveralls.io/builds/10803343)
Coverage increased (+1.0%) to 31.798% when pulling
Github user lizhanhui commented on a diff in the pull request:
https://github.com/apache/incubator-rocketmq/pull/79#discussion_r108344089
--- Diff: distribution/pom.xml ---
@@ -0,0 +1,125 @@
Github user Jaskey commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/75
@vongosling
Now nothing special has been done unless remove the hard corded aliyun code
snippet.
I will close this pr after merge.
---
If your project is set up for
Github user vongosling commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/75
@Jaskey Thanks for your elaborative consideration about exception. Let it
go as you have changed. we will merge this PR.
---
If your project is set up for it, you can reply to this
14 matches
Mail list logo