[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-30 Thread jeking3
Github user jeking3 closed the pull request at: https://github.com/apache/thrift/pull/466 --- 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 ena

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-30 Thread jeking3
Github user jeking3 commented on the pull request: https://github.com/apache/thrift/pull/466#issuecomment-97736680 As this was merged, closing the pull request. --- 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 proj

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
Github user jeking3 commented on the pull request: https://github.com/apache/thrift/pull/466#issuecomment-97620293 Wow, the build passed! I have an account on travis, but I have been submitting pull requests into the apache/thrift master, so I have to control them here as far as I kn

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread henrique
Github user henrique commented on the pull request: https://github.com/apache/thrift/pull/466#issuecomment-97618902 Hi James, As you do this on your own fork I thought you'd have the access to restart the Travis build manually, without the need of closing and opening the PR's. Hav

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
Github user jeking3 commented on the pull request: https://github.com/apache/thrift/pull/466#issuecomment-97616416 Sadly, Travis CI really isn't very stable. --- 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

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
Github user jeking3 closed the pull request at: https://github.com/apache/thrift/pull/466 --- 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 ena

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
GitHub user jeking3 reopened a pull request: https://github.com/apache/thrift/pull/466 THRIFT-3083 consolidate simple and threaded server run loops Currently the simple and threaded servers all share a very similar serve() workflow. This improvement story is to consolidate them and

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
Github user jeking3 commented on the pull request: https://github.com/apache/thrift/pull/466#issuecomment-97616177 package code.google.com/p/gomock/gomock: https://code.google.com/p/gomock/source/checkout?repo=: 502 Bad Gateway --- If your project is set up for it, you can reply to

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
GitHub user jeking3 reopened a pull request: https://github.com/apache/thrift/pull/466 THRIFT-3083 consolidate simple and threaded server run loops Currently the simple and threaded servers all share a very similar serve() workflow. This improvement story is to consolidate them and

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
Github user jeking3 closed the pull request at: https://github.com/apache/thrift/pull/466 --- 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 ena

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
Github user jeking3 commented on the pull request: https://github.com/apache/thrift/pull/466#issuecomment-97593757 Last build failed because of one make cross failure between java and go - two components I did not affect. --- If your project is set up for it, you can reply to this em

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
Github user jeking3 closed the pull request at: https://github.com/apache/thrift/pull/466 --- 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 ena

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
Github user jeking3 commented on the pull request: https://github.com/apache/thrift/pull/466#issuecomment-97396400 Giving a rebuild one more try in CI, if it fails then I'll rebase with master and see... --- If your project is set up for it, you can reply to this email and have your

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-29 Thread jeking3
GitHub user jeking3 reopened a pull request: https://github.com/apache/thrift/pull/466 THRIFT-3083 consolidate simple and threaded server run loops Currently the simple and threaded servers all share a very similar serve() workflow. This improvement story is to consolidate them and

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-28 Thread jeking3
GitHub user jeking3 reopened a pull request: https://github.com/apache/thrift/pull/466 THRIFT-3083 consolidate simple and threaded server run loops Currently the simple and threaded servers all share a very similar serve() workflow. This improvement story is to consolidate them and

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-28 Thread jeking3
Github user jeking3 commented on the pull request: https://github.com/apache/thrift/pull/466#issuecomment-97154526 There is a "make cross" error that is unrelated to my changes. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-28 Thread jeking3
Github user jeking3 closed the pull request at: https://github.com/apache/thrift/pull/466 --- 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 ena

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-28 Thread jeking3
Github user jeking3 closed the pull request at: https://github.com/apache/thrift/pull/466 --- 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 ena

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-28 Thread jeking3
GitHub user jeking3 reopened a pull request: https://github.com/apache/thrift/pull/466 THRIFT-3083 consolidate simple and threaded server run loops Currently the simple and threaded servers all share a very similar serve() workflow. This improvement story is to consolidate them and

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-28 Thread jeking3
GitHub user jeking3 reopened a pull request: https://github.com/apache/thrift/pull/466 THRIFT-3083 consolidate simple and threaded server run loops Currently the simple and threaded servers all share a very similar serve() workflow. This improvement story is to consolidate them and

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-28 Thread jeking3
Github user jeking3 closed the pull request at: https://github.com/apache/thrift/pull/466 --- 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 ena

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-28 Thread jeking3
Github user jeking3 closed the pull request at: https://github.com/apache/thrift/pull/466 --- 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 ena

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-28 Thread jeking3
GitHub user jeking3 reopened a pull request: https://github.com/apache/thrift/pull/466 THRIFT-3083 consolidate simple and threaded server run loops Currently the simple and threaded servers all share a very similar serve() workflow. This improvement story is to consolidate them and

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-27 Thread jeking3
Github user jeking3 closed the pull request at: https://github.com/apache/thrift/pull/466 --- 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 ena

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-27 Thread jeking3
GitHub user jeking3 reopened a pull request: https://github.com/apache/thrift/pull/466 THRIFT-3083 consolidate simple and threaded server run loops Currently the simple and threaded servers all share a very similar serve() workflow. This improvement story is to consolidate them and

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-27 Thread jeking3
Github user jeking3 commented on the pull request: https://github.com/apache/thrift/pull/466#issuecomment-96895697 Failed strangely - recycling. --- 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 hav

[GitHub] thrift pull request: THRIFT-3083 consolidate simple and threaded s...

2015-04-26 Thread jeking3
GitHub user jeking3 opened a pull request: https://github.com/apache/thrift/pull/466 THRIFT-3083 consolidate simple and threaded server run loops Currently the simple and threaded servers all share a very similar serve() workflow. This improvement story is to consolidate them and e