Skip to content

#126: Add support for Map type - #127

Merged
jozefbakus merged 2 commits into
masterfrom
feature/126-add-support-for-map-type
Oct 20, 2026
Merged

jozefbakus merged 2 commits into
masterfrom
feature/126-add-support-for-map-type

Conversation

@jozefbakus

@jozefbakus jozefbakus commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Overview

Map type was added to standardization

Release Notes

  • Added Map type support

Related

Closes #126

Test evidence:

Cannot add screenshots, but this version of standardization was tested on real pipeline/data
Resulting schema contains:
| | |-- metaData: map (nullable = true)
| | | |-- key: string
| | | |-- value: string (valueContainsNull = true)
And data with no errors in errCol

@github-actions

github-actions Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

JaCoCo code coverage report - scala 2.12.20, spark 3.5.7

Metric (instruction) Coverage Threshold Status
Overall 83.25% 80.0% ✅
Changed Files 91.43% 80.0% ✅
Report Coverage (O/Ch) Threshold (O/Ch) Status (O/Ch)
spark-data-standardization Jacoco Report - spark:3.5.7 scala:2.12.20 83.25% / 91.43% 80.0% / 80.0% ✅/✅
File Path Coverage Threshold Status
PlainSchemaGenerator.scala 91.06% 60.0% ✅
SchemaValidator.scala 78.8% 60.0% ✅
TypeParser.scala 93.77% 60.0% ✅
TypedStructField.scala 88.6% 60.0% ✅

Run 39249119798 · Event: pull_request

@github-actions

github-actions Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

JaCoCo code coverage report - scala 2.13.17, spark 4.1.2

Metric (instruction) Coverage Threshold Status
Overall 78.24% 75.0% ✅
Changed Files 86.04% 80.0% ✅
Report Coverage (O/Ch) Threshold (O/Ch) Status (O/Ch)
spark-data-standardization Jacoco Report - spark:4.1.2 scala:2.13.17 78.24% / 86.04% 75.0% / 80.0% ✅/✅
File Path Coverage Threshold Status
PlainSchemaGenerator.scala 89.91% 60.0% ✅
SchemaValidator.scala 80.19% 60.0% ✅
TypeParser.scala 85.54% 60.0% ✅
TypedStructField.scala 88.93% 60.0% ✅

Run 39249119798 · Event: pull_request

@yruslan yruslan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Checked the code and tests

@dk1844 dk1844 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (read the code)

@jozefbakus
jozefbakus merged commit cb4f559 into master Oct 20, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for Map type

3 participants