Skip to content

[SPARK-27195][SQL][TEST] Add AvroReadSchemaSuite#24135

Closed
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:SPARK-27195
Closed

[SPARK-27195][SQL][TEST] Add AvroReadSchemaSuite#24135
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:SPARK-27195

Conversation

@dongjoon-hyun
Copy link
Copy Markdown
Member

@dongjoon-hyun dongjoon-hyun commented Mar 18, 2019

What changes were proposed in this pull request?

The reader schema is said to be evolved (or projected) when it changed after the data is written by writers. Apache Spark file-based data sources have a test coverage for that, ReadSchemaSuite.scala. This PR aims to add AvroReadSchemaSuite to ensure the minimal consistency among file-based data sources and prevent a future regression in Avro data source.

How was this patch tested?

Pass the Jenkins with the newly added test suite.

@dongjoon-hyun
Copy link
Copy Markdown
Member Author

cc @gengliangwang

extends ReadSchemaSuite
with AddColumnIntoTheMiddleTest
with HideColumnInTheMiddleTest
with ChangePositionTest {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is just a AS-IS Apache Spark supportability in Avro data sources.

@dongjoon-hyun
Copy link
Copy Markdown
Member Author

Hi, @cloud-fan , @gatorsmile , @gengliangwang , @dbtsai .
Could you review this PR ? This adds the same test coverage for Avro data source.

@SparkQA
Copy link
Copy Markdown

SparkQA commented Mar 18, 2019

Test build #103629 has finished for PR 24135 at commit a7867d9.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyun
Copy link
Copy Markdown
Member Author

Thank you so much, @HyukjinKwon !
Merged to master.

@gengliangwang
Copy link
Copy Markdown
Member

A late LGTM 👍

@dongjoon-hyun
Copy link
Copy Markdown
Member Author

Thank you, @gengliangwang !

@dongjoon-hyun dongjoon-hyun deleted the SPARK-27195 branch March 19, 2019 03:42
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.

4 participants