Skip to content

refactor: remove 'incubating' branding, update docs & packaging#716

Open
contrueCT wants to merge 1 commit intoapache:masterfrom
contrueCT:chore/remove-incubating-branding
Open

refactor: remove 'incubating' branding, update docs & packaging#716
contrueCT wants to merge 1 commit intoapache:masterfrom
contrueCT:chore/remove-incubating-branding

Conversation

@contrueCT
Copy link

Purpose of the PR

  • close #xxx

Main Changes

  • update repo URLs, issue links, mailing list metadata, and docs references

  • remove the root DISCLAIMER

  • update NOTICE and release-docs headers from Apache HugeGraph(incubating) to Apache HugeGraph

  • rename distribution/package final.name outputs to non-incubating names

  • update dist/release scripts, Dockerfiles, CI/workflow paths, and README examples to use new package names

  • switch hugegraph-client-go module/import paths to github.com/apache/hugegraph-toolchain/hugegraph-client-go

  • intentionally kept third-party dependency/license references such as htrace-core4-*-incubating, since those are upstream artifact names rather than HugeGraph branding

Verifying these changes

  • Trivial rework / code cleanup without any test coverage. (No Need)
  • Already covered by existing tests, such as (please modify tests here).
  • Need tests and can be verified as follows:
    • xxx

Does this PR potentially affect the following parts?

  • Nope
  • Dependencies (add/update license info)
  • Modify configurations
  • The public API
  • Other affects (typed here)

Documentation Status

  • Doc - TODO
  • Doc - Done
  • Doc - No Need

@github-actions github-actions bot added hubble hugegraph-hubble loader hugegraph-loader client hugegraph-client tools hugegraph-tools spark labels Mar 19, 2026
@imbajin imbajin requested a review from Copilot March 19, 2026 06:33
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes “incubating” branding across the HugeGraph Toolchain repository by updating URLs, distribution/package naming, release artifacts, and related documentation so outputs align with post-incubation Apache naming.

Changes:

  • Update repository links, issue templates, badges, and documentation references from incubator-*/(incubating) to non-incubating locations and names.
  • Rename Maven final.name distribution artifacts to drop -incubating- and update packaging/release scripts and Dockerfiles accordingly.
  • Switch hugegraph-client-go module path/imports from github.com/apache/incubator-hugegraph-toolchain/... to github.com/apache/hugegraph-toolchain/....

Reviewed changes

Copilot reviewed 47 out of 47 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pom.xml Updates repo URL, mailing list address, artifact final name, and excludes for renamed hubble package paths
hugegraph-tools/pom.xml Drops -incubating- from tools artifact naming and updates module URL
hugegraph-tools/assembly/travis/install-hugegraph-from-source.sh Removes now-stale incubator TODO comments
hugegraph-spark-connector/pom.xml Drops -incubating- from spark connector artifact naming
hugegraph-spark-connector/assembly/travis/install-hugegraph-from-tar.sh Updates Apache download URL and extracted directory names to non-incubating HugeGraph tarball
hugegraph-spark-connector/assembly/travis/install-hugegraph-from-source.sh Removes now-stale incubator TODO comments
hugegraph-loader/pom.xml Drops -incubating- from loader artifact naming
hugegraph-loader/assembly/travis/install-hugegraph-from-tar.sh Removes now-stale incubator TODO comments
hugegraph-loader/assembly/travis/install-hugegraph-from-source.sh Removes now-stale incubator TODO comments
hugegraph-loader/README.md Updates download URLs and extracted directory names to non-incubating toolchain artifacts
hugegraph-loader/Dockerfile Updates COPY path glob to non-incubating loader directory
hugegraph-hubble/package.json Updates repository URL to non-incubator GitHub repo
hugegraph-hubble/hubble-dist/pom.xml Drops -incubating- from hubble distribution final name
hugegraph-hubble/hubble-dist/assembly/travis/download-hugegraph.sh Removes now-stale incubator TODO comment
hugegraph-hubble/README.md Updates badges, download commands, and build/run paths to non-incubating names
hugegraph-hubble/Dockerfile Updates COPY path glob to non-incubating hubble directory
hugegraph-dist/scripts/apache-release.sh Renames source/binary release artifacts and updates SVN dev path to non-incubator location
hugegraph-dist/release-docs/NOTICE Updates header branding and copyright year range
hugegraph-dist/release-docs/LICENSE Updates “Incubating” wording in subcomponents section
hugegraph-dist/pom.xml Drops -incubating- from dist artifact name and removes DISCLAIMER from packaged contents
hugegraph-client/src/main/java/org/apache/hugegraph/util/CommonUtil.java Updates HugeGraph doc repository URL prefix
hugegraph-client/assembly/travis/install-hugegraph-from-source.sh Removes now-stale incubator TODO comments
hugegraph-client-go/hugegraph.go Updates Go client imports to new module path
hugegraph-client-go/hgtransport/hgtransport.go Updates internal version import to new module path
hugegraph-client-go/go.mod Changes module path to github.com/apache/hugegraph-toolchain/hugegraph-client-go
hugegraph-client-go/api/v1/vertexlabel/vertexlabel_test.go Updates test imports to new module path
hugegraph-client-go/api/v1/vertexlabel/vertexlabel.go Updates imports to new module path
hugegraph-client-go/api/v1/vertex/vertex.go Updates imports to new module path
hugegraph-client-go/api/v1/version_test.go Updates test import to new module path
hugegraph-client-go/api/v1/version.go Updates imports to new module path
hugegraph-client-go/api/v1/schema_test.go Updates test import to new module path
hugegraph-client-go/api/v1/schema.go Updates imports to new module path
hugegraph-client-go/api/v1/propertykey/propertykey_test.go Updates test imports to new module path
hugegraph-client-go/api/v1/propertykey/propertykey.go Updates imports to new module path
hugegraph-client-go/api/v1/gremlin/gemlin_test.go Updates test import to new module path
hugegraph-client-go/api/v1/gremlin/gemlin.go Updates imports to new module path
hugegraph-client-go/api/v1/edgelabel/edgelabel_test.go Updates test imports to new module path
hugegraph-client-go/api/v1/edgelabel/edgelabel.go Updates imports to new module path
hugegraph-client-go/api/v1/api.go Updates imports to new module path
hugegraph-client-go/api/api.go Updates imports to new module path
hugegraph-client-go/README.md Updates go get and import examples to new module path
hugegraph-client-go/README.en.md Updates import examples to new module path
README.md Updates ecosystem links and hubble quick-start path to non-incubating names
NOTICE Updates header branding and copyright year range
DISCLAIMER Removes incubator DISCLAIMER file
.github/workflows/hubble-ci.yml Updates runtime directory glob to non-incubating hubble name
.github/ISSUE_TEMPLATE/feature_request.yml Updates issue search links to new repo location

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@imbajin
Copy link
Member

imbajin commented Mar 19, 2026

image

Record the loader-ci error & rerun

@contrueCT
Copy link
Author

image Record the `loader-ci` error & rerun

I checked the failing test case FileLoadTest.testSourceOrTargetPrimaryValueNull(). From the current code path, this looks more like a flaky test issue than a regression caused by this PR.

This PR only updates branding-related docs/metadata/package names/paths and does not change hugegraph-loader runtime logic.

For this test specifically, loader.load() is expected to throw, but loader.shutdown() is placed after it in the same lambda. So once load() fails, shutdown() is not executed, while vertex inserts are still submitted asynchronously. The test then immediately queries the graph and assumes all 7 vertices have already been flushed, which is not guaranteed. That would explain the observed expected:<7> but was:<5>.

I’ll take another look at whether we should stabilize this test separately, but at the moment this failure does not appear to be directly introduced by the incubating-branding cleanup in this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client hugegraph-client hubble hugegraph-hubble loader hugegraph-loader spark tools hugegraph-tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants