You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sedona.apache.org by GitBox <gi...@apache.org> on 2023/01/20 14:04:40 UTC

[GitHub] [sedona] umartin opened a new pull request, #749: [SEDONA-221] Outer join throws NPE for null geometries.

umartin opened a new pull request, #749:
URL: https://github.com/apache/sedona/pull/749

   
   ## Did you read the Contributor Guide?
   
   - Yes, I have read [Contributor Rules](https://sedona.apache.org/community/rule/) and [Contributor Development Guide](https://sedona.apache.org/community/develop/)
   
   ## Is this PR related to a JIRA ticket?
   
   - Yes, the URL of the assoicated JIRA ticket is https://issues.apache.org/jira/browse/SEDONA-221. The PR name follows the format `[SEDONA-XXX] my subject`.
   
   
   ## What changes were proposed in this PR?
   
   This fixes the issues in broadcast left and anti joins with null geometries.
   
   Rows are mapped to (Geometry, UnsafeRow) to allow rows with null geometries to be emitted.
   Conditions for left and anti joins, where nulls can happen, checks for null.
   
   This patch also adds a SQLMetric for number of output row visible in spark-ui. All native Spark join exec have them
   
   ![image](https://user-images.githubusercontent.com/1275096/213715460-8d623437-a072-41e0-a1e7-cded26a05988.png)
   
   
   ## How was this patch tested?
   
   Tests added
   
   ## Did this PR include necessary documentation updates?
   
   - No, this PR does not affect any public API so no need to change the docs.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [sedona] jiayuasu merged pull request #749: [SEDONA-221] Outer join throws NPE for null geometries.

Posted by GitBox <gi...@apache.org>.
jiayuasu merged PR #749:
URL: https://github.com/apache/sedona/pull/749


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org