torodb / stampede

The ToroDB solution to provide better analytics on top of MongoDB and make it easier to migrate from MongoDB to SQL
https://www.torodb.com/stampede/
GNU Affero General Public License v3.0
1.77k stars 120 forks source link

Changed logger adquisition from Log4j LogManager to our own LoggerFac… #159

Closed gortiz closed 7 years ago

gortiz commented 7 years ago

…tory

This is used to be able to inject specific loggers on bundles that need to be used multiple times, like Mongo Core and Mongo Repl

codecov-io commented 7 years ago

Codecov Report

Merging #159 into sharding will increase coverage by 1.2%. The diff coverage is 36.33%.

@@              Coverage Diff              @@
##             sharding     #159     +/-   ##
=============================================
+ Coverage       17.12%   18.33%   +1.2%     
- Complexity       1412     1511     +99     
=============================================
  Files             769      780     +11     
  Lines           26847    26803     -44     
  Branches         2337     2337             
=============================================
+ Hits             4597     4913    +316     
+ Misses          21926    21556    -370     
- Partials          324      334     +10
Impacted Files Coverage Δ Complexity Δ
...ava/com/torodb/mongodb/guice/MongodCoreModule.java 100% <ø> (ø) 2 <0> (ø) :x:
...ds/signatures/diagnostic/ListDatabasesCommand.java 4.76% <ø> (+4.76%) 2 <0> (+2) :white_check_mark:
.../repl/impl/AlwaysConsistentConsistencyHandler.java 20% <ø> (ø) 1 <0> (ø) :x:
...odb/mongodb/repl/oplogreplier/ReplSyncFetcher.java 0% <0%> (ø) 0 <0> (ø) :x:
...a/com/torodb/backend/AbstractDbBackendService.java 0% <0%> (ø) 0 <0> (ø) :x:
...a/com/torodb/backend/meta/SnapshotUpdaterImpl.java 0% <0%> (ø) 0 <0> (ø) :x:
...com/torodb/backend/meta/AbstractSchemaUpdater.java 0% <0%> (ø) 0 <0> (ø) :x:
...ain/java/com/torodb/core/d2r/D2RLoggerFactory.java 0% <0%> (ø) 0 <0> (?)
...in/java/com/torodb/backend/BackendServiceImpl.java 0% <0%> (ø) 0 <0> (ø) :x:
...odb/mongodb/repl/oplogreplier/ReplSyncApplier.java 0% <0%> (ø) 0 <0> (ø) :x:
... and 153 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update eb22305...7560233. Read the comment docs.