Skip to content

Conversation

@qxzzxq
Copy link
Member

@qxzzxq qxzzxq commented Jan 27, 2022

No description provided.

dependabot bot and others added 2 commits December 3, 2021 03:02
Bumps [delta-core_2.12](https://github.com/delta-io/delta) from 0.7.0 to 1.1.0. - [Release notes](https://github.com/delta-io/delta/releases) - [Commits](delta-io/delta@v0.7.0...v1.1.0) --- updated-dependencies: - dependency-name: io.delta:delta-core_2.12 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
@codecov
Copy link

codecov bot commented Jan 27, 2022

Codecov Report

Merging #234 (03eb5bf) into master (40ef056) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@ Coverage Diff @@ ## master #234 +/- ## ========================================== + Coverage 97.89% 97.95% +0.05%  ========================================== Files 61 61 Lines 1999 2000 +1 Branches 129 134 +5 ========================================== + Hits 1957 1959 +2  + Misses 42 41 -1 
Flag Coverage Δ
master_2.11_2.4 ?
master_2.12_3.0 ?
pr_2.11_2.3 92.65% <50.00%> (?)
pr_2.11_2.4 97.60% <100.00%> (?)
pr_2.12_2.4 97.58% <100.00%> (?)
pr_2.12_3.0 97.73% <100.00%> (?)
pr_2.12_3.2 97.73% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...thub/setl/storage/repository/SparkRepository.scala 98.33% <100.00%> (ø)
...cala/io/github/setl/config/FileConnectorConf.scala 100.00% <0.00%> (ø)
...o/github/setl/storage/SparkRepositoryBuilder.scala 98.30% <0.00%> (+1.69%) ⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

In the newer Spark version, column name with no space character will no longer have the '`' character. Therefore changes were made to have the unit test running correctly
@qxzzxq qxzzxq changed the title Fix Delta issue WIP Fix Delta issue Jan 27, 2022
@qxzzxq qxzzxq marked this pull request as draft February 21, 2022 17:14
@qxzzxq qxzzxq marked this pull request as ready for review March 12, 2022 23:43
@qxzzxq qxzzxq requested review from JorisTruong and maroil March 13, 2022 00:10
@qxzzxq qxzzxq merged commit 467220f into SETL-Framework:master Mar 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants