----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39407/ -----------------------------------------------------------
Review request for geode, anilkumar gingade, Jason Huynh, Jianxia Chen, and Lynn Gallinat. Repository: geode Description ------- This introduces a subclass of the JGroups UDP protocol in order to force it to create daemon threads. It also inhibits setting time-to-live on the UDP socket unless multicast is enabled. This was causing InvocationTargetExceptions to be logged at ERROR level on Windows 7 machines. Diffs ----- gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/DistributionMessage.java 8cefd9c8ecddaf649a60d0d2576696b675c9cb31 gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/AddressManager.java a1305d54da1607d65ac03b8991daca37a9f45232 gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/JGroupsMessenger.java 556f974278c8ab260246331fab672c25e87e4fe3 gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/Transport.java PRE-CREATION gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/DistributedCacheOperation.java bff0ff750ab2f96b686bd345633eb3f54f8e3d6d gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java bea7bd1f00c9f05b20eb23f40fe7a906b051b087 gemfire-core/src/main/resources/com/gemstone/gemfire/distributed/internal/jgroups-config.xml 73e66b43bf782fb7fc449757951cbd72cd85aa8e gemfire-core/src/main/resources/com/gemstone/gemfire/distributed/internal/jgroups-mcast.xml 4c6f25c5177c5aed1784402e177ae3da273bc9b2 gemfire-core/src/test/java/com/gemstone/gemfire/cache/client/internal/AutoConnectionSourceWithUDPDUnitTest.java 9f1d67a118134b7eb6abf2bb761cdabec0d68bd0 gemfire-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/JGroupsMessengerJUnitTest.java 985cfb54166da7f1bcaccc301e172c3d5c5e8395 Diff: https://reviews.apache.org/r/39407/diff/ Testing ------- unit testing (3 unrelated failures), external integration testing Thanks, Bruce Schuchardt
