-
Notifications
You must be signed in to change notification settings - Fork 692
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
[SEDONA-120] Add ST_BuildArea function and tests #624
Conversation
extends UnaryGeometryExpression with CodegenFallback { | ||
assert(inputExpressions.length == 1) | ||
|
||
override protected def nullSafeEval(geometry: Geometry): Any = { |
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 this be a nullable function?
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.
Yes, this should be a nullable function. So no need to extend UnaryGeometryExpression
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.
Have updated the function and added null checks
b59e630
to
597f861
Compare
@jiayuasu Have fixed the test failures, seems to be a transient issue with the CI |
2978fd1
to
8722ccf
Compare
Did you read the Contributor Guide?
Is this PR related to a JIRA ticket?
[SEDONA-120] Add ST_BuildArea function and tests
.What changes were proposed in this PR?
How was this patch tested?
Did this PR include necessary documentation updates?