Spell the groupId the way the project spells itself - #53
Merged
estebanzimanyi merged 1 commit intoAug 31, 2026
Merged
Conversation
`pom.xml` declares `<groupId>org.mobiltydb</groupId>`, dropping the second `i`, while every Java source in the tree declares `package org.mobilitydb.spark` and the generator writes into `.../org/mobilitydb/spark/generated`. The coordinate disagreed with the code it names. Changing it costs nothing to anyone: the pom carries no `distributionManagement` and no deploy plugin, so the artifact is never published, and a code search across JMEOS, MobilityFlink, MobilityKafka and MEOS-API finds no reference to `org.mobiltydb` — 0 in each, against 2 for `SparkMeos` inside this repository. No Java source mentions it either; the only other occurrences are in the README, in a package name that belongs to sources this tree does not have.
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
1 participant
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.
pom.xmldeclares<groupId>org.mobiltydb</groupId>, dropping the secondi,while every Java source in the tree declares
package org.mobilitydb.sparkand thegenerator writes into
.../org/mobilitydb/spark/generated. The coordinatedisagreed with the code it names.
Changing it costs nothing to anyone: the pom carries no
distributionManagementand no deploy plugin, so the artifact is never published, and a code search across
JMEOS, MobilityFlink, MobilityKafka and MEOS-API finds no reference to
org.mobiltydb— 0 in each, against 2 forSparkMeosinside this repository. NoJava source mentions it either; the only other occurrences are in the README, in a
package name that belongs to sources this tree does not have.