Skip to content

Add OSM PBF reader.#1819

Closed
Imbruced wants to merge 10 commits into
masterfrom
osm-reader
Closed

Add OSM PBF reader.#1819
Imbruced wants to merge 10 commits into
masterfrom
osm-reader

Conversation

@Imbruced
Copy link
Copy Markdown
Member

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

  • Yes, the URL of the associated JIRA ticket is https://issues.apache.org/jira/browse/SEDONA-XXX. The PR name follows the format [SEDONA-XXX] my subject.

  • No:

    • this is a documentation update. The PR name follows the format [DOCS] my subject
    • this is a CI update. The PR name follows the format [CI] my subject

What changes were proposed in this PR?

How was this patch tested?

Did this PR include necessary documentation updates?

  • Yes, I am adding a new API. I am using the current SNAPSHOT version number in vX.Y.Z format.
  • Yes, I have updated the documentation.
  • No, this PR does not affect any public API so no need to change the documentation.

@github-actions github-actions Bot added the root label Feb 17, 2025
@Imbruced
Copy link
Copy Markdown
Member Author

@jiayuasu, I'm unsure if that's how we want to handle shading the proto. OSM uses proto four, and I think Hadoop is still version 3. I can remove the reader from the spark common, move it to the OSM module, add proto 4, and exclude it from the spark common, as I am already doing.

@Imbruced
Copy link
Copy Markdown
Member Author

I'll create the Jira ticket + add the documentation update. If we agree on implementation, then I'll also copy the code for spark 3.4 and 3.3

@Imbruced Imbruced closed this Feb 20, 2025
@jiayuasu jiayuasu deleted the osm-reader branch February 28, 2025 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant