-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[Feature][transform] Add a module to set default value for null field #1958
Conversation
...sform-spark-etl/src/main/scala/org/apache/seatunnel/spark/transform/service/EtlHandler.scala
Outdated
Show resolved
Hide resolved
Please add an issue and describe why you need to add this transform plugin. In fact, I am confusion with the plugin name |
After I commit new code,Dead Link Checker failed,It doesn't seem to be my problem. Can you help me rerun the ci |
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.
Generally looks good to me, It might be better if you can submit a PR to add e2e case for this plugin.
Please fix the scala checkstyle problem. |
OK, I will revise it and submit it later :) |
|
Sorry, I accidentally clicked the close button, could you please help to restart the build :) |
|
Sorry, this is a problem with my local docker installation, which has been solved. |
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
#1963
Purpose of this pull request
Set default value for null field
Check list
New License Guide