-
Notifications
You must be signed in to change notification settings - Fork 236
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
Add Partial Delta Lake Support for Databricks 13.3 #9644
Conversation
Signed-off-by: Raza Jafri <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Partial review, have not gotten through all the files yet.
...common/src/main/databricks/scala/com/nvidia/spark/rapids/delta/DatabricksDeltaProvider.scala
Show resolved
Hide resolved
...ake/common/src/main/databricks/scala/com/nvidia/spark/rapids/delta/shims/package-shims.scala
Outdated
Show resolved
Hide resolved
.../org/apache/spark/sql/execution/datasources/v2/rapids/GpuAtomicCreateTableAsSelectExec.scala
Outdated
Show resolved
Hide resolved
...org/apache/spark/sql/execution/datasources/v2/rapids/GpuAtomicReplaceTableAsSelectExec.scala
Outdated
Show resolved
Hide resolved
.../org/apache/spark/sql/execution/datasources/v2/rapids/GpuAtomicCreateTableAsSelectExec.scala
Outdated
Show resolved
Hide resolved
...org/apache/spark/sql/execution/datasources/v2/rapids/GpuAtomicReplaceTableAsSelectExec.scala
Outdated
Show resolved
Hide resolved
...rk341db/src/main/scala/com/databricks/sql/transaction/tahoe/rapids/GpuMergeIntoCommand.scala
Outdated
Show resolved
Hide resolved
...rk341db/src/main/scala/com/databricks/sql/transaction/tahoe/rapids/GpuMergeIntoCommand.scala
Outdated
Show resolved
Hide resolved
...db/src/main/scala/com/databricks/sql/transaction/tahoe/rapids/GpuOptimisticTransaction.scala
Outdated
Show resolved
Hide resolved
...db/src/main/scala/com/databricks/sql/transaction/tahoe/rapids/GpuOptimisticTransaction.scala
Outdated
Show resolved
Hide resolved
...-lake/delta-spark341db/src/main/scala/com/nvidia/spark/rapids/delta/shims/DeltaLogShim.scala
Outdated
Show resolved
Hide resolved
...-lake/delta-spark341db/src/main/scala/com/nvidia/spark/rapids/delta/shims/DeltaLogShim.scala
Outdated
Show resolved
Hide resolved
Signed-off-by: Raza Jafri <[email protected]>
This reverts commit e2fd85c.
This reverts commit 7a87438.
This reverts commit 6c1259d.
...lake/common/src/main/delta-io/scala/org/apache/spark/sql/delta/rapids/DeltaRuntimeShim.scala
Outdated
Show resolved
Hide resolved
...rk341db/src/main/scala/com/databricks/sql/transaction/tahoe/rapids/GpuMergeIntoCommand.scala
Outdated
Show resolved
Hide resolved
...rk341db/src/main/scala/com/databricks/sql/transaction/tahoe/rapids/GpuMergeIntoCommand.scala
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's here looks OK, but seems like there's things that are missing, as the changes ni #9677 imply there's a lot of tests still failing after this.
build |
…replace_table_as_select
premerge will fail until #9688 is merged |
Build |
build |
This pull request aims to introduce support for Delta Lake on Databricks 13.3, enabling enhanced data management capabilities within the Databricks ecosystem. The integration of Delta Lake is essential for ensuring robust data handling and efficient data processing functionalities.
Known Issues:
Changes Made:
Delta Lake Integration: The PR includes the necessary modifications to seamlessly integrate Delta Lake functionality into the Databricks 13.3 environment, allowing users to leverage advanced data management features such as version control and transactional capabilities.
Contributes towards issue 9175