-
Notifications
You must be signed in to change notification settings - Fork 325
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
Support Spark 4.0 #2854
Support Spark 4.0 #2854
Conversation
baf30ec
to
88915a1
Compare
1456811
to
e660c65
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #2854 +/- ##
=======================================
Coverage 85.52% 85.52%
=======================================
Files 58 58
Lines 3413 3413
=======================================
Hits 2919 2919
Misses 494 494 ☔ View full report in Codecov by Sentry. |
e660c65
to
612487c
Compare
91aba83
to
42f9e67
Compare
bc29477
to
7f75063
Compare
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.
Few things to look at. Besides looking good.
integration/spark/app/src/main/java/io/openlineage/spark/agent/OpenLineageSparkListener.java
Outdated
Show resolved
Hide resolved
private static final String SPARK_3 = "(3.*)"; | ||
private static final String SPARK_3_3 = "(3\\.[3-9].*)"; | ||
private static final String GREATER_THAN_SPARK2 = "(3.*)"; | ||
private static final String GREATER_THAN_SPARK3_3 = "(3\\.[3-9].*)"; |
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.
Should those both contain Spark 4? If not, why?
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.
As mentioned few lines above:
// TODO: Please note the test remains disabled for Spark 4.0 for now (no applicable connector
// version available)
Signed-off-by: Pawel Leszczynski <leszczynski.pawel@gmail.com>
Signed-off-by: Pawel Leszczynski <leszczynski.pawel@gmail.com>
Signed-off-by: Pawel Leszczynski <leszczynski.pawel@gmail.com>
7f75063
to
2c970a5
Compare
Support Spark 4.0 preview -> https://spark.apache.org/news/spark-4.0.0-preview1.html