Skip to content

Make SparkWriteBuilder and SparkWrite classes public - #4

Open
daksha121 wants to merge 11 commits into
masterfrom
spark.write.public.classes
Open

Make SparkWriteBuilder and SparkWrite classes public#4
daksha121 wants to merge 11 commits into
masterfrom
spark.write.public.classes

Conversation

@daksha121

Copy link
Copy Markdown
Owner

We have some scenarios for which we need to support opinionated writes to Iceberg tables using Spark.
In an attempt to support those, we plan to extend the SparkWriteBuilder and SparkWrite classes. One example of the scenarios we are trying to achieve:

  • We want to be able to allow incremental writes only to the Iceberg table. If the above classes were made public, we could extend them and block all actions except incremental writes. Either based on a custom table property or even by default.

Please let us know if there are alternative ways for us to support such functionality using Iceberg.

@daksha121 daksha121 changed the title Spark.write.public.classes Make SparkWriteBuilder and SparkWrite classes public Jul 16, 2021
Assertions.assertThat(actual).containsExactlyInAnyOrderElementsOf(Iterables.concat(expected));
}

@SuppressWarnings("unchecked")

@SreeramGarlapati SreeramGarlapati Jul 19, 2021

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this lookslike as if this is a new change. I guess your master is not up to date.

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.

2 participants