Remove log4j1 from bk test code
authorZhangJian He <shoothzj@gmail.com>
Tue, 19 Apr 2022 16:34:50 +0000 (00:34 +0800)
committerGitHub <noreply@github.com>
Tue, 19 Apr 2022 16:34:50 +0000 (09:34 -0700)
commit760b9e0192e954ebcd7b3b302f3b43672da21a58
treecfb3558a083b717e3bc27610022ace54c5e181a6
parent7c8930330da3c20747206ef101a497dc58ab0547
Remove log4j1 from bk test code

### Motivation
this is a part work of remove log4j1 from bookeeper, first remove it from our test code, unit test and integrate test

### Changes
- remove log4j1 scope test dependency
- use log4j2.xml instread of log4j.properties
- introduce testtools module to reduce duplication,  if a module needs particular configuration (very unlikely) we can add a log4j2-test.xml file which has precedence over log4j2.xml one. idea from [nicoloboschi](https://github.com/nicoloboschi)

Reviewers: Andrey Yegorov <None>, Enrico Olivelli <eolivelli@gmail.com>, Nicolò Boschi <boschi1997@gmail.com>

This closes #3208 from Shoothzj/remove-log4j1-from-bk-test-code2
70 files changed:
bookkeeper-benchmark/pom.xml
bookkeeper-benchmark/src/test/resources/log4j.properties [deleted file]
bookkeeper-common/build.gradle
bookkeeper-common/pom.xml
bookkeeper-common/src/test/java/org/apache/bookkeeper/common/util/TestOrderedExecutorDecorators.java
bookkeeper-common/src/test/resources/log4j.properties [deleted file]
bookkeeper-dist/src/main/resources/LICENSE-all.bin.txt
bookkeeper-dist/src/main/resources/LICENSE-bkctl.bin.txt
bookkeeper-dist/src/main/resources/LICENSE-server.bin.txt
bookkeeper-http/http-server/pom.xml
bookkeeper-server/build.gradle
bookkeeper-server/pom.xml
bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/LedgerStorageCheckpointTest.java
bookkeeper-server/src/test/java/org/apache/bookkeeper/client/MdcContextTest.java
bookkeeper-server/src/test/java/org/apache/bookkeeper/meta/AbstractZkLedgerManagerTest.java
bookkeeper-server/src/test/resources/log4j.properties [deleted file]
bookkeeper-slogger/slf4j/pom.xml
bookkeeper-stats/pom.xml
circe-checksum/build.gradle
circe-checksum/pom.xml
circe-checksum/src/test/resources/log4j.properties [deleted file]
dependencies.gradle
metadata-drivers/etcd/build.gradle
metadata-drivers/etcd/pom.xml
metadata-drivers/etcd/src/test/resources/log4j.properties [deleted file]
microbenchmarks/pom.xml
pom.xml
settings.gradle
stream/clients/java/all/build.gradle
stream/clients/java/all/pom.xml
stream/clients/java/all/src/test/resources/log4j.properties [deleted file]
stream/clients/java/base/build.gradle
stream/clients/java/base/pom.xml
stream/clients/java/base/src/test/resources/log4j.properties [deleted file]
stream/clients/java/kv/build.gradle
stream/clients/java/kv/pom.xml
stream/clients/java/kv/src/test/resources/log4j.properties [deleted file]
stream/distributedlog/core/build.gradle
stream/distributedlog/core/pom.xml
stream/distributedlog/core/src/test/resources/log4j.properties [deleted file]
stream/distributedlog/protocol/build.gradle
stream/distributedlog/protocol/pom.xml
stream/distributedlog/protocol/src/test/resources/log4j.properties [deleted file]
stream/proto/build.gradle
stream/proto/pom.xml
stream/proto/src/test/resources/log4j.properties [deleted file]
stream/server/build.gradle
stream/server/pom.xml
stream/server/src/test/resources/log4j.properties [deleted file]
stream/statelib/build.gradle
stream/statelib/pom.xml
stream/statelib/src/test/resources/log4j.properties [deleted file]
stream/storage/impl/build.gradle
stream/storage/impl/pom.xml
stream/storage/impl/src/test/resources/log4j.properties [deleted file]
tests/integration-tests-base/pom.xml
tests/integration-tests-utils/src/main/resources/log4j.properties [deleted file]
tests/integration-tests-utils/src/main/resources/log4j2.xml [moved from metadata-drivers/etcd/src/test/resources/log4j2.xml with 81% similarity]
tests/integration/cluster/build.gradle
tests/integration/cluster/pom.xml
tests/integration/cluster/src/test/resources/log4j.properties [deleted file]
tests/integration/standalone/build.gradle
tests/integration/standalone/pom.xml
tests/integration/standalone/src/test/resources/log4j.properties [deleted file]
testtools/build.gradle [new file with mode: 0644]
testtools/pom.xml [new file with mode: 0644]
testtools/src/main/resources/log4j2.xml [new file with mode: 0644]
tools/ledger/build.gradle
tools/ledger/pom.xml
tools/ledger/src/test/resources/log4j.properties [deleted file]