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
Pass the SparkSession when resolving the tables in the DeltaTableBuilder #1476
Pass the SparkSession when resolving the tables in the DeltaTableBuilder #1476
Conversation
Signed-off-by: Helge Bruegner <helge@bruegner.de>
Signed-off-by: Helge Bruegner <helge@bruegner.de>
I would need some guidance regarding the testing: is there any test I could add or is this covered already? |
I would need some guidance regarding the testing: is there any test I could add or is this covered already? It's fine to not add a test for this trivial change. |
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.
LGTM. Thanks!
(Cherry-pick 68c8e18 to branch-2.0) Signed-off-by: Helge Bruegner <helge@bruegner.de> - Forward the SparkSession when resolving the tables in the DeltaTableBuilder - Remove some unused imports Resolves delta-io#1475 / No. Closes delta-io#1476 Signed-off-by: Shixiong Zhu <zsxwing@gmail.com> GitOrigin-RevId: 880de0f55ee79289cecd19d74c4c177c76d30aeb
(Cherry-pick 68c8e18 to branch-2.0) Signed-off-by: Helge Bruegner <helge@bruegner.de> - Forward the SparkSession when resolving the tables in the DeltaTableBuilder - Remove some unused imports Resolves delta-io#1475 / No. Closes delta-io#1476 Signed-off-by: Shixiong Zhu <zsxwing@gmail.com> GitOrigin-RevId: 880de0f55ee79289cecd19d74c4c177c76d30aeb
(Cherry-pick 68c8e18 to branch-2.1) Signed-off-by: Helge Bruegner <helge@bruegner.de> - Forward the SparkSession when resolving the tables in the DeltaTableBuilder - Remove some unused imports Resolves #1475 / No. Closes #1476 Signed-off-by: Shixiong Zhu <zsxwing@gmail.com> GitOrigin-RevId: 880de0f55ee79289cecd19d74c4c177c76d30aeb
Signed-off-by: Helge Bruegner helge@bruegner.de
Description
Resolves #1475
How was this patch tested?
/
Does this PR introduce any user-facing changes?
No.