Skip to content

Conversation

@MaxGekk
Copy link
Member

@MaxGekk MaxGekk commented Aug 12, 2020

What changes were proposed in this pull request?

Why are the changes needed?

  • To improve test coverage, and test other row SerDe - org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe.
  • The removed test is not needed anymore because the bug reported in SPARK-32594 is triggered by the TEXTFILE file format too.

Does this PR introduce any user-facing change?

No

How was this patch tested?

By running the modified test suite HiveSerDeReadWriteSuite.

@MaxGekk
Copy link
Member Author

MaxGekk commented Aug 12, 2020

@cloud-fan @HyukjinKwon Please, review this PR.

@SparkQA
Copy link

SparkQA commented Aug 12, 2020

Test build #127376 has finished for PR 29417 at commit 5e1415d.

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

@HyukjinKwon
Copy link
Member

Merged to master and branch-3.0.

HyukjinKwon pushed a commit that referenced this pull request Aug 12, 2020
…eReadWriteSuite`

### What changes were proposed in this pull request?
- Test TEXTFILE together with the PARQUET and ORC file formats in `HiveSerDeReadWriteSuite`
- Remove the "SPARK-32594: insert dates to a Hive table" added by #29409

### Why are the changes needed?
- To improve test coverage, and test other row SerDe - `org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe`.
- The removed test is not needed anymore because the bug reported in SPARK-32594 is triggered by the TEXTFILE file format too.

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
By running the modified test suite `HiveSerDeReadWriteSuite`.

Closes #29417 from MaxGekk/textfile-HiveSerDeReadWriteSuite.

Authored-by: Max Gekk <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
(cherry picked from commit f664aaa)
Signed-off-by: HyukjinKwon <[email protected]>
@MaxGekk MaxGekk deleted the textfile-HiveSerDeReadWriteSuite branch December 11, 2020 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants