Get data for Mapping Saturation indicator from ohsomeDB#994
Open
matthiasschaub wants to merge 6 commits intomainfrom
Open
Get data for Mapping Saturation indicator from ohsomeDB#994matthiasschaub wants to merge 6 commits intomainfrom
matthiasschaub wants to merge 6 commits intomainfrom
Conversation
cc0ff71 to
9072a1f
Compare
|
Hagellach37
reviewed
Jan 29, 2026
| @@ -0,0 +1,55 @@ | |||
| -- Parsing the GeoJSON directly in the WHERE clause instead of | |||
Collaborator
There was a problem hiding this comment.
as @Gigaszi has proven that this is not always the case, we could also remove this comment here. :)
Member
Author
There was a problem hiding this comment.
Then I would like to parse it in the WITH statement because it makes the query more readable.
Member
Author
There was a problem hiding this comment.
Or should we leave it like this in the cases it which it does matter? But then I would like to keep the comment otherwise I will refactor it in a couple of moths when I forgot it :D
9072a1f to
842f444
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



No description provided.