Skip to content

[WIP] showacase PR - strange spark behaviour#11891

Closed
homar wants to merge 1 commit intotrinodb:masterfrom
homar:homar/spark_caches_stuff_iceberg
Closed

[WIP] showacase PR - strange spark behaviour#11891
homar wants to merge 1 commit intotrinodb:masterfrom
homar:homar/spark_caches_stuff_iceberg

Conversation

@homar
Copy link
Member

@homar homar commented Apr 9, 2022

Description

DO NOT MERGE

Strange spark behaviour:
testSparkDoesntReadTrinoTableWithDeletes - fails because spark doesn't see updates made by trino after it performs a delete
testSparkReadsTrinoTableWithDeletes - doesn't fail, the difference is that updates after delete are made by spark.

It looks like spark caches something internally. It has bad implications as it will make operations like optimize, delete_orphan_files and expire_snapshots break future queries from spark

Related issues, pull requests, and links

Documentation

( ) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

( ) No release notes entries required.
( ) Release notes entries required with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Apr 9, 2022
@homar homar force-pushed the homar/spark_caches_stuff_iceberg branch from 93533c3 to db1bae9 Compare April 9, 2022 08:29
@homar homar force-pushed the homar/spark_caches_stuff_iceberg branch 2 times, most recently from 6ef33c3 to c70f037 Compare April 12, 2022 10:18
@homar homar force-pushed the homar/spark_caches_stuff_iceberg branch from c70f037 to dc6c458 Compare April 12, 2022 13:39
@homar
Copy link
Member Author

homar commented Apr 13, 2022

I am closing this one as its only purpose was to demonstrate strange behaviour that was caused by spark caching metadata internally

@homar homar closed this Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant