1 more thing... I would additionally advise rewording the sentence...
*> Please add log4j-core to the classpath.* To read... "*Please consider adding log4j-core, or another Logging provider (e.g. Logback), to your classpath. Apache Geode works best with Log4j.*" Food for thought. -John On Tue, Sep 3, 2019 at 10:40 AM John Blum <jb...@pivotal.io> wrote: > +1 > > Ran SDG build against Apache Geode 1.9.1 build snapshots (for RC3). > > However, can we seriously reconsider logging the follow message at ERROR? > Ugh! > > ERROR StatusLogger Log4j2 could not find a logging implementation. Please > add log4j-core to the classpath. Using SimpleLogger to log to the console... > > WARN is more than sufficient. If no logging provider is on the CLASSPATH, > it creates a lot of noise! > > -John > > > On Fri, Aug 30, 2019 at 12:27 PM Dave Barnes <dbar...@pivotal.io> wrote: > >> +1 >> Checked the docs: Successfully built viewed the Geode User Guide and the >> Javadocs. >> >> On Fri, Aug 30, 2019 at 11:11 AM Owen Nichols <onich...@pivotal.io> >> wrote: >> >> > Hello Geode dev community, >> > >> > This is a release candidate for Apache Geode, version 1.9.1.RC3. >> > Thanks to all the community members for their contributions to this >> > release! >> > >> > Please do a review and give your feedback. The deadline is 3PM PST Thu, >> > September 05 2019. >> > Release notes can be found at: >> > >> https://cwiki.apache.org/confluence/display/GEODE/Release+Notes#ReleaseNotes-1.9.1 >> > >> > Please note that we are voting upon the source tags: rel/v1.9.1.RC3 >> > >> > Apache Geode: >> > https://github.com/apache/geode/tree/rel/v1.9.1.RC3 >> > Apache Geode examples: >> > https://github.com/apache/geode-examples/tree/rel/v1.9.1.RC3 >> > Apache Geode native: >> > https://github.com/apache/geode-native/tree/rel/v1.9.1.RC3 >> > >> > Source and binary files: >> > https://dist.apache.org/repos/dist/dev/geode/1.9.1.RC3/ >> > >> > Maven staging repo: >> > https://repository.apache.org/content/repositories/orgapachegeode-1057 >> > >> > Geode's KEYS file containing PGP keys we use to sign the release: >> > https://github.com/apache/geode/blob/develop/KEYS >> > >> > PS: Command to run geode-examples: ./gradlew -PgeodeReleaseUrl= >> > https://dist.apache.org/repos/dist/dev/geode/1.9.1.RC3 >> > -PgeodeRepositoryUrl= >> > https://repository.apache.org/content/repositories/orgapachegeode-1057 >> > build runAll >> > >> > Regards >> > Owen & Kirk >> > >> > >> > > > -- > -John > john.blum10101 (skype) > -- -John john.blum10101 (skype)
