Github user lizhanhui commented on the issue:
https://github.com/apache/incubator-rocketmq/pull/53
This feature is definitely nice to have and it expands scenarios that
RocketMQ fits best. For example, RocketMQ may have close, if not better,
performance with Kafka in log collecting usage.
That said, this is a pretty important feature and it matters so much that
we need to get it right at the beginning. We'd better have a design document
first, then discuss various impacts it brings about in the mailing list.
As for this PR, it's generally good, yet, still needs more work: code
duplication, message validation logic discrepancy, excessive constraints on
usages(No delay messages, messages of a batch must have same topic, etc) and
previously mentioned compatibility issue with older brokers.
---
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 the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
