Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1105] CDF + Vacuum tests #1177

Closed

Conversation

scottsand-db
Copy link
Collaborator

See the project plan at #1105.

This PR adds two tests to DeltaVacuumSuite testing CDF + VACUUM integration.

GitOrigin-RevId: 130fa5e5e53bb62b0e58b01b1cd188ec662627cb
@scottsand-db scottsand-db added the enhancement New feature or request label Jun 4, 2022
@scottsand-db scottsand-db self-assigned this Jun 4, 2022
Copy link
Collaborator

@vkorukanti vkorukanti left a comment

Choose a reason for hiding this comment

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

LGTM

mmengarelli pushed a commit to mmengarelli/delta.io that referenced this pull request Jun 8, 2022
GitOrigin-RevId: ceacc3a6239f36e76cc8cf4f3447285bb06fc6a0

CDF + Vacuum tests

This PR adds two tests testing CDF + VACUUM integration.

Closes delta-io#1177

GitOrigin-RevId: f2f7b187cb3cc78c267d378eaf3d0657a56241d9

Adds miscellaneous (e.g. end-to-end workload, and CDCReader) tests for CDF.

Resolves delta-io#1178

GitOrigin-RevId: 5c7da4ff9413d84e73137a80673872065de8267b

Minor refactor to UpdateCommand

GitOrigin-RevId: ab3fcbe0522aa194ac0781730a50112655d5c7ec

 Fixes delta-io#348 Support Dynamic Partition Overwrite

The goal of this PR to to support dynamic partition overwrite mode on writes to delta.

To enable this on a per write add `.option("partitionOverwriteMode", "dynamic")`. It can also be set per sparkSession in the SQL Config using `.config("spark.sql.sources.partitionOverwriteMode", "dynamic")`.

Some limitations of this pullreq:
Dynamic partition overwrite mode in combination with replaceWhere is not supported. If both are set this will result in an error.
The SQL `INSERT OVERWRITE` syntax does not yet support dynamic partition overwrite. For this more changes will be needed to be made to `org.apache.spark.sql.delta.catalog.DeltaTableV2` and related classes.

Fixes delta-io#348
Closes delta-io#371

Signed-off-by: Allison Portis <allison.portis@databricks.com>
GitOrigin-RevId: 5b01e5b04e573dabe91ac2d71991a127617b8038

Add Checkpoint + CDF test.

This PR adds a test to ensure CDC fields are not included in the checkpoint file.

Resolves delta-io#1180

GitOrigin-RevId: d4a7b8bc4d1a79d30806ff18c5b507f6edcd964c
@vkorukanti vkorukanti added this to the 2.0 milestone Jun 28, 2022
jbguerraz pushed a commit to jbguerraz/delta that referenced this pull request Jul 6, 2022
This PR adds two tests testing CDF + VACUUM integration.

Closes delta-io#1177

GitOrigin-RevId: f2f7b187cb3cc78c267d378eaf3d0657a56241d9
jbguerraz pushed a commit to jbguerraz/delta that referenced this pull request Jul 6, 2022
This PR adds two tests testing CDF + VACUUM integration.

Closes delta-io#1177

GitOrigin-RevId: f2f7b187cb3cc78c267d378eaf3d0657a56241d9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants