-
Notifications
You must be signed in to change notification settings - Fork 11
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
Fix/hof structs combined error and improve resolution of unresolved columns #227
Conversation
d1b305d
to
f3339fe
Compare
Codecov Report
@@ Coverage Diff @@
## main #227 +/- ##
==========================================
+ Coverage 96.39% 96.83% +0.44%
==========================================
Files 53 53
Lines 831 946 +115
Branches 9 11 +2
==========================================
+ Hits 801 916 +115
Misses 30 30
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
core/src/test/spark_3.0_3.1_3.2/scala/doric/syntax/MapColumns3xSpec.scala
Show resolved
Hide resolved
Still pending some optimizations in creating the lambdas |
80c1000
to
a82b36c
Compare
…lects. CurrentDate now is resolved in creation.
…"asDoric" in some basic cases.
a82b36c
to
e00578f
Compare
@eruizalo Ready for your review 😄 |
core/src/test/spark_3.0_3.1_3.2/scala/doric/syntax/ArrayColumns3xSpec.scala
Outdated
Show resolved
Hide resolved
Co-authored-by: Eduardo Ruiz Alonso <edu.r26@gmail.com>
Co-authored-by: Eduardo Ruiz Alonso <edu.r26@gmail.com>
Co-authored-by: Eduardo Ruiz Alonso <edu.r26@gmail.com>
Added zipper to improve zipWith syntax
HOF functions for arrays and maps couldn't be used combined with Row children due to problems with lambda column references.
Added also flatten for collections of collections
Added currentDate alternative that allows setting the Return type other than Date
Fixes #225