Closed alexjbush closed 5 years ago
:exclamation: No coverage uploaded for pull request base (
develop@7d148f3
). Click here to learn what that means. The diff coverage is100%
.
@@ Coverage Diff @@
## develop #88 +/- ##
==========================================
Coverage ? 82.62%
==========================================
Files ? 60
Lines ? 1658
Branches ? 71
==========================================
Hits ? 1370
Misses ? 288
Partials ? 0
Impacted Files | Coverage Δ | |
---|---|---|
...a/waimak/configuration/CaseClassConfigParser.scala | 97.26% <ø> (ø) |
|
...ration/PropertiesFilePropertyProviderBuilder.scala | 100% <ø> (ø) |
|
...w/spark/CacheAsParquetConfigurationExtension.scala | 100% <100%> (ø) |
|
...ala/com/coxautodata/waimak/dataflow/DataFlow.scala | 96.61% <100%> (ø) |
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 7d148f3...5bce087. Read the comment docs.
Description
This PR introduces configuration-driven extension functionality.
This type of extension adds a pre-execution hook when an extension is enabled by setting
spark.waimak.dataflow.extensions=${extensionKey},otherextension
.Instances of the extension trait must be registered services in the
META-INF/services
file as they are loaded usingServiceLoader
.It also provides an implementation of a Cache As Parquet extension:
To allow this to work, the configuration utils (i.e.
CaseClassConfigParser
) have now been moved into core.Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Unit tests