[GitHub] zookeeper pull request #427: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket i...

2017-12-13 Thread fr0stbyte
Github user fr0stbyte commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/427#discussion_r156860284 --- Diff: src/c/configure.ac --- @@ -130,18 +130,66 @@ main() else exit(0); } -], AC_MSG_RESULT(yes) - ipv6=yes

[GitHub] zookeeper issue #427: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defin...

2017-12-12 Thread fr0stbyte
Github user fr0stbyte commented on the issue: https://github.com/apache/zookeeper/pull/427 @phunt thanks, will keep that in mind. ---

[GitHub] zookeeper issue #427: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defin...

2017-12-12 Thread fr0stbyte
Github user fr0stbyte commented on the issue: https://github.com/apache/zookeeper/pull/427 @phunt @mfenes updated the initial comment of this PR. Let me know if you need me to move it into a separate comment ---

[GitHub] zookeeper issue #427: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defin...

2017-12-09 Thread fr0stbyte
Github user fr0stbyte commented on the issue: https://github.com/apache/zookeeper/pull/427 @anmolnar sorry, my editor is configured to strip the trailing white spaces . Do you need me to disable that? ---

[GitHub] zookeeper pull request #427: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket i...

2017-12-07 Thread fr0stbyte
GitHub user fr0stbyte opened a pull request: https://github.com/apache/zookeeper/pull/427 [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defined @phunt @anmolnar PR for 3.4 branch. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] zookeeper issue #410: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defin...

2017-12-05 Thread fr0stbyte
Github user fr0stbyte commented on the issue: https://github.com/apache/zookeeper/pull/410 @phunt @anmolnar any suggestion on changes or does this look good ? ---

[GitHub] zookeeper issue #410: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defin...

2017-11-27 Thread fr0stbyte
Github user fr0stbyte commented on the issue: https://github.com/apache/zookeeper/pull/410 @phunt Let me know if the changes are inline with what you've had in mind. Once we clear that, I will add the changes for 3.6 ---

[GitHub] zookeeper issue #410: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defin...

2017-11-21 Thread fr0stbyte
Github user fr0stbyte commented on the issue: https://github.com/apache/zookeeper/pull/410 @phunt both options seem fine to me. I think would prefer the first one, that way if the library remains ABI compatible, the clients won't need to recompile. ---

[GitHub] zookeeper issue #410: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defin...

2017-11-15 Thread fr0stbyte
Github user fr0stbyte commented on the issue: https://github.com/apache/zookeeper/pull/410 @phunt I'd have no problem making in backward compatible, but I am not sure how. Can you offer some guidance here ? @anmolnar In the description of MESOS-4065 you can see 2 processes

[GitHub] zookeeper issue #410: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defin...

2017-11-01 Thread fr0stbyte
Github user fr0stbyte commented on the issue: https://github.com/apache/zookeeper/pull/410 @afine Thanks for looking at the PR Here is the bug tracked on Mesos side : https://issues.apache.org/jira/browse/MESOS-4065 ---

[GitHub] zookeeper pull request #410: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket i...

2017-10-27 Thread fr0stbyte
GitHub user fr0stbyte opened a pull request: https://github.com/apache/zookeeper/pull/410 [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defined You can merge this pull request into a Git repository by running: $ git pull https://github.com/fr0stbyte/zookeeper ZOOKEEPER

[GitHub] zookeeper pull request #407: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket i...

2017-10-27 Thread fr0stbyte
Github user fr0stbyte closed the pull request at: https://github.com/apache/zookeeper/pull/407 ---

[GitHub] zookeeper pull request #407: [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket i...

2017-10-25 Thread fr0stbyte
GitHub user fr0stbyte opened a pull request: https://github.com/apache/zookeeper/pull/407 [ZOOKEEPER-2338] - set SOCK_CLOEXEC on socket if defined This addresses the issue in https://issues.apache.org/jira/browse/ZOOKEEPER-2338 preventing the sockets to leak into the forked