Conversation
Replace references to 'incubator'/'incubating' across the repo to the final 'hugegraph' branding and packaging names. Updates include: removal of DISCLAIMER, NOTICE year bump, README and docs link/badge fixes, pom final.name changes (remove -incubating suffix), Dockerfile and assembly scripts adjusted to new package paths, numerous docs and tests updated to new URLs/paths, mailing list/contact updates, and minor serena project.yml additions.
There was a problem hiding this comment.
Pull request overview
Refactors repository branding and distribution naming from incubator/“incubating” to final Apache HugeGraph naming, updating packaging outputs, release tooling, Docker build paths, and documentation links accordingly.
Changes:
- Drop
-incubatingsuffix from distributionfinal.namevalues and update scripts/tests/docs that reference extracted package directories. - Update repository/docs URLs and mailing list addresses from incubator-era locations to the final HugeGraph locations.
- Remove
DISCLAIMERand update NOTICE/Licensing docs to reflect post-incubation branding/year.
Reviewed changes
Copilot reviewed 36 out of 36 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| pom.xml | Update developer/mailing-list metadata to non-incubator addresses/branding |
| install-dist/scripts/apache-release.sh | Update release artifact names and SVN dist path (remove incubator/) |
| install-dist/release-docs/NOTICE | Remove “(incubating)” branding + bump copyright year |
| install-dist/release-docs/LICENSE | Remove “(Incubating)” wording in subcomponents section |
| install-dist/pom.xml | Rename dist final.name and update copied dist folder names |
| hugegraph-store/pom.xml | Remove -incubating suffix from store distribution name |
| hugegraph-store/docs/operations-guide.md | Update example tarball/dir names to non-incubating |
| hugegraph-store/docs/development-guide.md | Update working dir and output name examples |
| hugegraph-store/docs/deployment-guide.md | Update tarball/dir names and example paths |
| hugegraph-store/README.md | Update distribution path examples to non-incubating |
| hugegraph-store/Dockerfile | Update COPY glob to match new dist directory naming |
| hugegraph-server/pom.xml | Remove -incubating suffix from server distribution name |
| hugegraph-server/hugegraph-dist/src/assembly/travis/start-store.sh | Update store dir name used in CI scripts |
| hugegraph-server/hugegraph-dist/src/assembly/travis/start-pd.sh | Update PD dir name used in CI scripts |
| hugegraph-server/hugegraph-dist/src/assembly/travis/run-api-test.sh | Update server dir name + doc URL for downloaded binary |
| hugegraph-server/hugegraph-dist/src/assembly/travis/run-api-test-for-raft.sh | Update server dir name used in CI script |
| hugegraph-server/hugegraph-dist/pom.xml | Update reference link from incubator repo to final repo |
| hugegraph-server/hugegraph-dist/docker/README.md | Update example.groovy link to final repo |
| hugegraph-server/Dockerfile-hstore | Update COPY glob to match new dist directory naming |
| hugegraph-server/Dockerfile | Update COPY glob to match new dist directory naming |
| hugegraph-pd/pom.xml | Remove -incubating suffix from PD distribution name |
| hugegraph-pd/Dockerfile | Update COPY glob to match new dist directory naming |
| hugegraph-pd/AGENTS.md | Update output/dir name examples to non-incubating |
| hugegraph-commons/pom.xml | Update developer/mailing-list metadata to non-incubator addresses/branding |
| hugegraph-commons/hugegraph-common/src/test/java/org/apache/hugegraph/unit/rest/RestClientTest.java | Update test download URL to final docs repo |
| hugegraph-commons/hugegraph-common/pom.xml | Update project URL to final repo location |
| hugegraph-commons/README.md | Update badges/links and image URLs to final repo locations |
| hugegraph-cluster-test/pom.xml | Remove -incubating suffix from CT distribution name |
| hugegraph-cluster-test/hugegraph-clustertest-test/src/main/java/org/apache/hugegraph/SimpleClusterTest/BaseSimpleTest.java | Update documented CT directory name |
| hugegraph-cluster-test/hugegraph-clustertest-test/src/main/java/org/apache/hugegraph/MultiClusterTest/BaseMultiClusterTest.java | Update documented CT directory name |
| hugegraph-cluster-test/hugegraph-clustertest-minicluster/src/main/java/org/apache/hugegraph/ct/base/ClusterConstant.java | Update package prefix constants to non-incubating |
| README.md | Update CI badges, ecosystem links, download URLs, contributors link |
| NOTICE | Remove “(incubating)” branding + bump copyright year |
| DISCLAIMER | Remove incubator DISCLAIMER file |
| CONTRIBUTING.md | Update fork/clone/upstream and other links from incubator repo to final repo |
| .serena/project.yml | Add/expand Serena project configuration placeholders |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 36 out of 36 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # Build distribution package only | ||
| mvn clean package -pl hg-pd-dist -am -DskipTests | ||
| # Output: hg-pd-dist/target/apache-hugegraph-pd-incubating-<version>.tar.gz | ||
| # Output: hg-pd-dist/target/apache-hugegraph-pd-<version>.tar.gz |
There was a problem hiding this comment.
The documented output location hg-pd-dist/target/apache-hugegraph-pd-<version>.tar.gz doesn’t match the hg-pd-dist build configuration: the assembly is configured to place the tarball in the parent hugegraph-pd/ directory (outputDirectory=${dist.dir} and tar ... ${dist.dir}/${final.name}.tar.gz). Please update this path so the instructions reflect the actual build output.
| # Output: hg-pd-dist/target/apache-hugegraph-pd-<version>.tar.gz | |
| # Output: hugegraph-pd/apache-hugegraph-pd-<version>.tar.gz |
| @@ -35,7 +35,7 @@ mvn package -Dmaven.test.skip=true -ntp | |||
| wget -P $SERVER_DIR/lib/ https://repo1.maven.org/maven2/mysql/mysql-connector-java/8.0.28/mysql-connector-java-8.0.28.jar | |||
|
|
|||
| if [[ ! -e "$SERVER_DIR/ikanalyzer-2012_u6.jar" ]]; then | |||
There was a problem hiding this comment.
This existence check is looking for ikanalyzer-2012_u6.jar in $SERVER_DIR, but the wget -P command saves it under $SERVER_DIR/lib/. As written, the condition will always be true (unless a different file happens to exist), causing an unnecessary re-download on every run and extra network flakiness. Update the check to test for the file in the same directory it’s downloaded to.
| if [[ ! -e "$SERVER_DIR/ikanalyzer-2012_u6.jar" ]]; then | |
| if [[ ! -e "$SERVER_DIR/lib/ikanalyzer-2012_u6.jar" ]]; then |
| - Main class: `org.apache.hugegraph.store.node.StoreNodeApplication` | ||
| - VM options: `-Xms4g -Xmx4g -Dconfig.file=conf/application.yml` | ||
| - Working directory: `hugegraph-store/hg-store-dist/target/apache-hugegraph-store-incubating-1.7.0` | ||
| - Working directory: `hugegraph-store/hg-store-dist/target/apache-hugegraph-store-1.7.0` |
There was a problem hiding this comment.
The documented working directory hugegraph-store/hg-store-dist/target/apache-hugegraph-store-1.7.0 doesn’t match the actual assembly output location. hg-store-dist is configured to write the assembled directory into the parent hugegraph-store/ directory (via outputDirectory=${dist.dir}), so this path will likely not exist after packaging and will break the run configuration instructions. Please update the doc to point at the real assembled directory path.
| - Working directory: `hugegraph-store/hg-store-dist/target/apache-hugegraph-store-1.7.0` | |
| - Working directory: `hugegraph-store/apache-hugegraph-store-1.7.0` |
| **Start Server**: | ||
| ```bash | ||
| cd hugegraph-store/hg-store-dist/target/apache-hugegraph-store-incubating-1.7.0 | ||
| cd hugegraph-store/hg-store-dist/target/apache-hugegraph-store-1.7.0 |
There was a problem hiding this comment.
The “Start Server” example cd hugegraph-store/hg-store-dist/target/apache-hugegraph-store-1.7.0 appears to point to a non-existent directory given the current hg-store-dist build config (assembly output is written to hugegraph-store/, not hg-store-dist/target). This will cause the documented start commands to fail; please update the path to the actual assembled distribution directory.
| cd hugegraph-store/hg-store-dist/target/apache-hugegraph-store-1.7.0 | |
| cd hugegraph-store/apache-hugegraph-store-1.7.0 |
| mvn clean package -DskipTests | ||
|
|
||
| # Output: hg-store-dist/target/apache-hugegraph-store-incubating-<version>.tar.gz | ||
| # Output: hg-store-dist/target/apache-hugegraph-store-<version>.tar.gz |
There was a problem hiding this comment.
The documented distribution output path hg-store-dist/target/apache-hugegraph-store-<version>.tar.gz doesn’t match the build: the hg-store-dist module writes the tarball into the parent hugegraph-store/ directory (see outputDirectory=${dist.dir} and tar ... ${dist.dir}/${final.name}.tar.gz). Please update the doc so users can find the produced tarball.
| # Output: hg-store-dist/target/apache-hugegraph-store-<version>.tar.gz | |
| # Output: hugegraph-store/apache-hugegraph-store-<version>.tar.gz |
Replace references to 'incubator'/'incubating' across the repo to the final 'hugegraph' branding and packaging names. Updates include: removal of DISCLAIMER, NOTICE year bump, README and docs link/badge fixes, pom final.name changes (remove -incubating suffix),
Dockerfile and assembly scripts adjusted to new package paths, numerous docs and tests updated to new URLs/paths, mailing list/contact updates