[GitHub] incubator-distributedlog pull request #47: The CONFIG.ini file has no purpos...

2016-11-30 Thread franckcuny
GitHub user franckcuny opened a pull request: https://github.com/apache/incubator-distributedlog/pull/47 The CONFIG.ini file has no purpose outside of Twitter You can merge this pull request into a Git repository by running: $ git pull https://github.com/franckcuny/incubator-d

[GitHub] incubator-distributedlog pull request #45: Build cobertura report only on ma...

2016-11-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-distributedlog/pull/45 --- 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 th

incubator-distributedlog git commit: Build cobertura report only on master for successful builds

2016-11-30 Thread fcuny
Repository: incubator-distributedlog Updated Branches: refs/heads/master ebaf14580 -> e611ef9e8 Build cobertura report only on master for successful builds Building the report takes time, and when someone submits a pull request, we should not show it as failure if the build timeouts for the re

Re: Add DistributedLog IO

2016-11-30 Thread Sijie Guo
Thanks Khurrum. I will try to take a look. (Although I am not an expert of Beam.) It would also be awesome if you can share your experience on building this via soms blog posts, once you finished this. Sijie On Nov 30, 2016 4:02 AM, "Khurrum Nasim" wrote: > FYI > > I just sent a pull request f

Re: Add DistributedLog IO

2016-11-30 Thread Khurrum Nasim
FYI I just sent a pull request for adding a bounded source to Beam for reading distributedlog streams. I am going to send out the pull request for adding an unbounded source and a sink after that. If you are interested in this and willing to help review it, this is the pull request - https://gith