[GitHub] bahir-flink issue #4: [BAHIR-59][AMQ] Fix constructor visibility and error m...

2016-09-19 Thread lresende
Github user lresende commented on the issue: https://github.com/apache/bahir-flink/pull/4 Looks like this has already been merged by @tedyu in a105a7c3455b9851f951506ba91f1472002d323f @rmetzger Could you please close the pr. --- If your project is set up for it, you can repl

[GitHub] bahir-flink issue #4: [BAHIR-59][AMQ] Fix constructor visibility and error m...

2016-09-19 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/bahir-flink/pull/4 LGTM, +1 to merge --- 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] bahir-flink issue #4: [BAHIR-59][AMQ] Fix constructor visibility and error m...

2016-09-06 Thread tedyu
Github user tedyu commented on the issue: https://github.com/apache/bahir-flink/pull/4 lgtm --- 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 featu