[GitHub] [sedona] jiayuasu opened a new pull request, #746: [SEDONA-230] rdd.saveAsGeoJSON should generate feature properties with field names

2023-01-05 Thread GitBox
jiayuasu opened a new pull request, #746: URL: https://github.com/apache/sedona/pull/746 ## Did you read the Contributor Guide? - Yes, I have read [Contributor Rules](https://sedona.apache.org/community/rule/) and [Contributor Development

[jira] [Commented] (SEDONA-229) Linter Messages in Build Log

2023-01-05 Thread Jia Yu (Jira)
[ https://issues.apache.org/jira/browse/SEDONA-229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17655256#comment-17655256 ] Jia Yu commented on SEDONA-229: --- [~dougdennis] Yes, I 100% support this! > Linter Messages in Build Log >

[jira] [Created] (SEDONA-230) rdd.saveAsGeoJSON should generate feature properties with field names

2023-01-05 Thread Jia Yu (Jira)
Jia Yu created SEDONA-230: - Summary: rdd.saveAsGeoJSON should generate feature properties with field names Key: SEDONA-230 URL: https://issues.apache.org/jira/browse/SEDONA-230 Project: Apache Sedona

[GitHub] [sedona] jiayuasu merged pull request #742: [SEDONA-223] Add ST_Split

2023-01-05 Thread GitBox
jiayuasu merged PR #742: URL: https://github.com/apache/sedona/pull/742 -- 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:

[GitHub] [sedona] jiayuasu commented on pull request #742: [SEDONA-223] Add ST_Split

2023-01-05 Thread GitBox
jiayuasu commented on PR #742: URL: https://github.com/apache/sedona/pull/742#issuecomment-1373023036 Awesome. Will merge. -- 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.

Re: JoinQueryRaw.SpatialJoinQueryFlat for polygon - linestring join?

2023-01-05 Thread Jia Yu
And, I believe you should be able to post things directly if you join the Sedona community server. On Thu, Jan 5, 2023 at 4:44 PM Jia Yu wrote: > Hi Mark, > > Sedona supports polygon-linestring joins. Did you set > 'ConsiderBoundaryIntersection' to true? See: >

Fwd: JoinQueryRaw.SpatialJoinQueryFlat for polygon - linestring join?

2023-01-05 Thread Jia Yu
Hi Mark, Sedona supports polygon-linestring joins. Did you set 'ConsiderBoundaryIntersection' to true? See: https://sedona.apache.org/1.3.1-incubating/tutorial/core-python/#write-a-spatial-join-query This is the last parameter in Sedona Python JoinQueryRaw.SpatialJoinQueryFlat(). Thanks, Jia

[GitHub] [sedona] Kimahriman commented on a diff in pull request #742: [SEDONA-223] Add ST_Split

2023-01-05 Thread GitBox
Kimahriman commented on code in PR #742: URL: https://github.com/apache/sedona/pull/742#discussion_r1062985739 ## common/src/main/java/org/apache/sedona/common/utils/GeometrySplitter.java: ## @@ -0,0 +1,331 @@ +/** + * Licensed under the Apache License, Version 2.0 (the

[GitHub] [sedona] neontty commented on pull request #744: [SEDONA-156] Support spatial filter push-down for GeoParquetFileFormat

2023-01-05 Thread GitBox
neontty commented on PR #744: URL: https://github.com/apache/sedona/pull/744#issuecomment-1372809028 @Kontinuation your code is so clean! Thank you for putting in this work. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [sedona] jiayuasu commented on pull request #745: [SEDONA-227] Python Serde Refactor

2023-01-05 Thread GitBox
jiayuasu commented on PR #745: URL: https://github.com/apache/sedona/pull/745#issuecomment-1372700871 @Kontinuation @Imbruced Please feel free to chime in with comments :-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [sedona] jiayuasu commented on pull request #743: [SEDONA-228] Standardize logging modules

2023-01-05 Thread GitBox
jiayuasu commented on PR #743: URL: https://github.com/apache/sedona/pull/743#issuecomment-1372632857 Great explanation! Thank you! -- 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

[jira] [Commented] (SEDONA-156) predicate pushdown support for GeoParquet

2023-01-05 Thread RJ Marcus (Jira)
[ https://issues.apache.org/jira/browse/SEDONA-156?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17655105#comment-17655105 ] RJ Marcus commented on SEDONA-156: -- Jia is correct; my work schedule was reprioritized and I'm not

[GitHub] [sedona] douglasdennis commented on a diff in pull request #742: [SEDONA-223] Add ST_Split

2023-01-05 Thread GitBox
douglasdennis commented on code in PR #742: URL: https://github.com/apache/sedona/pull/742#discussion_r1062589421 ## common/src/main/java/org/apache/sedona/common/utils/GeometrySplitter.java: ## @@ -0,0 +1,331 @@ +/** + * Licensed under the Apache License, Version 2.0 (the

[jira] [Created] (SEDONA-229) Linter Messages in Build Log

2023-01-05 Thread Doug Dennis (Jira)
Doug Dennis created SEDONA-229: -- Summary: Linter Messages in Build Log Key: SEDONA-229 URL: https://issues.apache.org/jira/browse/SEDONA-229 Project: Apache Sedona Issue Type: Improvement

[GitHub] [sedona] Kimahriman commented on a diff in pull request #742: [SEDONA-223] Add ST_Split

2023-01-05 Thread GitBox
Kimahriman commented on code in PR #742: URL: https://github.com/apache/sedona/pull/742#discussion_r1062394602 ## common/src/main/java/org/apache/sedona/common/utils/GeometrySplitter.java: ## @@ -0,0 +1,331 @@ +/** + * Licensed under the Apache License, Version 2.0 (the

[GitHub] [sedona] Kimahriman commented on pull request #743: [SEDONA-228] Standardize logging modules

2023-01-05 Thread GitBox
Kimahriman commented on PR #743: URL: https://github.com/apache/sedona/pull/743#issuecomment-1372116194 Do you mean for tests or for real use? For tests the logs should all go to `target/unit-tests.log`. Copied this from Spark so the stdout/err of the tests is less verbose and mostly

[GitHub] [sedona] douglasdennis opened a new pull request, #745: [SEDONA-227] Python Serde Refactor

2023-01-05 Thread GitBox
douglasdennis opened a new pull request, #745: URL: https://github.com/apache/sedona/pull/745 ## Did you read the Contributor Guide? - Yes, I have read [Contributor Rules](https://sedona.apache.org/community/rule/) and [Contributor Development

[jira] [Commented] (SEDONA-219) Github checks are failing due to ubuntu version for buildĀ 

2023-01-05 Thread awadhesh tanwar (Jira)
[ https://issues.apache.org/jira/browse/SEDONA-219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17654885#comment-17654885 ] awadhesh tanwar commented on SEDONA-219: [~jiayu] ST_ConcaveHull builds passes even with the

[GitHub] [sedona] douglasdennis commented on pull request #742: [SEDONA-223] Add ST_Split

2023-01-05 Thread GitBox
douglasdennis commented on PR #742: URL: https://github.com/apache/sedona/pull/742#issuecomment-1371906517 @jiayuasu Added it in. Let me know if I was supposed to do it some other way. The output I see looks like this: `23/01/04 23:52:03 ERROR GeometrySplitter: Colinear sections