Based on my reading of [1] and [2] I don’t think the comment is necessary and should be removed. Any other opinions?
Anthony [1] http://www.apache.org/legal/src-headers.html [2] http://www.apache.org/dev/apply-license.html#contributor-copyright > On Jul 26, 2016, at 12:38 AM, doribd <g...@git.apache.org> wrote: > > Github user doribd commented on a diff in the pull request: > > https://github.com/apache/incubator-geode/pull/199#discussion_r72201660 > > --- Diff: > geode-client-native/src/cppcache/impl/ThinClientRedundancyManager.hpp --- > @@ -56,7 +56,7 @@ namespace gemfire { > void netDown( ); > void acquireRedundancyLock( ) { m_redundantEndpointsLock.acquire_read( > ); } > void releaseRedundancyLock( ) { m_redundantEndpointsLock.release( ); } > - volatile bool allEndPointDiscon() { return m_IsAllEpDisCon;} > + bool allEndPointDiscon() { return m_IsAllEpDisCon;} //Copyright: > Amdocs Software Systems Limited, 2016 > --- End diff -- > > Are we required to remove this line? > > > --- > 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 enabled but not working, please > contact infrastructure at infrastruct...@apache.org or file a JIRA ticket > with INFRA. > ---
signature.asc
Description: Message signed with OpenPGP using GPGMail
