Github user manuzhang commented on the issue:
https://github.com/apache/incubator-gearpump/pull/198
@yanghua This can be closed now.
---
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 featu
Github user manuzhang commented on the issue:
https://github.com/apache/incubator-gearpump/pull/198
@yanghua @huafengw
I pushed the changes to the `state` branch such that we can collaborate on
this.
---
If your project is set up for it, you can reply to this email and have your
Github user manuzhang commented on the issue:
https://github.com/apache/incubator-gearpump/pull/198
@yanghua Thanks, this is a great effort. I left several comments after a
quick glance through.
More generally,
1. This is a big PR with new interfaces being introduced
Github user manuzhang commented on the issue:
https://github.com/apache/incubator-gearpump/pull/198
R: @manuzhang @huafengw
---
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
enabl
Github user codecov-io commented on the issue:
https://github.com/apache/incubator-gearpump/pull/198
#
[Codecov](https://codecov.io/gh/apache/incubator-gearpump/pull/198?src=pr&el=h1)
Report
> Merging
[#198](https://codecov.io/gh/apache/incubator-gearpump/pull/198?src=pr&el=desc)
