More notes: 1) Preserve joptsimple authors (njadhav, Nikhil Jadhav, Emils Solmanis, Paul Holger)
2) Do not remove the author tag from MX4JModelMBean (there’s a separate bug on the source header for this file). 3) Do not remove the author tag from ANSIBuffer. Again, this is a third-party work where we should not modify unnecessarily. 4) Preserve the author tags on ObjectIntHashMap. 5) Do not remove the author tag from ObjIdConcurrentMap. This also needs to have the Apache source header removed as it is in the public domain. Should be added to LICENSE as well. 6) Do not remove the author tag from ForceableLinkedBlockingQueue. This also needs to have the Apache source header removed as it is in the public domain. Should be added to LICENSE as well. 7) Do not remove the author tag from CompactConcurrentHashSet2. This also needs to have the Apache source header removed as it is in the public domain. Should be added to LICENSE as well. Whew. I guess it’s the day for massive commits! Anthony > On Feb 19, 2016, at 1:55 PM, metatype <[email protected]> wrote: > > Github user metatype commented on a diff in the pull request: > > https://github.com/apache/incubator-geode/pull/100#discussion_r53524020 > > --- Diff: > extensions/gemfire-modules-session/src/test/java/com/gemstone/gemfire/modules/session/junit/SeparateClassloaderTestRunner.java > --- > @@ -23,7 +23,6 @@ > import java.net.URLClassLoader; > > /** > - * @author StackOverflow > --- End diff -- > > We need to investigate the origin of this code to ensure it is safe to > include in the geode code base. > > > --- > 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 [email protected] or file a JIRA ticket > with INFRA. > ---
signature.asc
Description: Message signed with OpenPGP using GPGMail
