Skip to content

airframe-parquet: Scala 3 support#2211

Merged
xerial merged 3 commits intomasterfrom
parquet-scala3-part2
Jun 1, 2022
Merged

airframe-parquet: Scala 3 support#2211
xerial merged 3 commits intomasterfrom
parquet-scala3-part2

Conversation

@xerial
Copy link
Copy Markdown
Member

@xerial xerial commented Jun 1, 2022

No description provided.

@xerial xerial linked an issue Jun 1, 2022 that may be closed by this pull request
airframe-parquet: Scala 3 support #2209
Closed
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 1, 2022

Codecov Report

Merging #2211 (8c42cc8) into master (90931d9) will decrease coverage by 0.00%.
The diff coverage is 86.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2211      +/-   ##
==========================================
- Coverage   82.67%   82.67%   -0.01%     
==========================================
  Files         313      313              
  Lines       12381    12390       +9     
  Branches      791      802      +11     
==========================================
+ Hits        10236    10243       +7     
- Misses       2145     2147       +2
Impacted Files Coverage Δ
...main/scala/wvlet/airframe/codec/MessageCodec.scala 83.87% <ø> (ø)
...c/main/scala/wvlet/airframe/control/Resource.scala 75.00% <0.00%> (-15.00%) ⬇️
...ala/wvlet/airframe/parquet/ParquetWriteCodec.scala 93.10% <ø> (ø)
...n/scala/wvlet/airframe/parquet/RecordBuilder.scala 87.50% <ø> (ø)
...a/wvlet/airframe/parquet/ParquetObjectWriter.scala 79.56% <50.00%> (+0.22%) ⬆️
...in/scala/wvlet/airframe/codec/PrimitiveCodec.scala 84.92% <100.00%> (+0.16%) ⬆️
...cala/wvlet/airframe/codec/ScalaStandardCodec.scala 97.60% <100.00%> (ø)
...rc/main/scala/wvlet/airframe/parquet/Parquet.scala 100.00% <100.00%> (ø)
...a/wvlet/airframe/parquet/ParquetRecordWriter.scala 83.33% <100.00%> (+3.33%) ⬆️
.../wvlet/airframe/parquet/ParquetWriterAdapter.scala 100.00% <100.00%> (ø)

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 90931d9...8c42cc8. Read the comment docs.

@xerial
Copy link
Copy Markdown
Member Author

xerial commented Jun 1, 2022

  • AnyCodec now accepts knownSurfaces parameter to resolve Surface from Class[_] at runtime
  • Add Resource.use for using loan pattern
  • Parquet.newRecordWriter accepts knownSurface parameter to support nested objects

@xerial xerial merged commit 8ef9535 into master Jun 1, 2022
@xerial xerial mentioned this pull request Jun 1, 2022
Closed
@xerial xerial deleted the parquet-scala3-part2 branch June 2, 2022 16:22
@xerial xerial mentioned this pull request Sep 20, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

airframe-parquet: Scala 3 support

1 participant

@xerial