imbajin commented on code in PR #3116:
URL: https://github.com/apache/hugegraph/pull/3116#discussion_r3652775334
##########
install-dist/release-docs/LICENSE:
##########
@@ -364,11 +350,8 @@ non-Apache-2.0 license text is required for a bundled
component.
https://central.sonatype.com/artifact/com.taobao.arthas/arthas-packaging/3.7.1
-> Apache 2.0
https://central.sonatype.com/artifact/com.vaadin.external.google/android-json/0.0.20131108.vaadin1
-> Apache 2.0
https://central.sonatype.com/artifact/commons-beanutils/commons-beanutils/1.9.4
-> Apache 2.0
- https://central.sonatype.com/artifact/commons-cli/commons-cli/1.1 ->
Apache 2.0
Review Comment:
⚠️ `commons-cli-1.5.0.jar` is still bundled, but this change removes the
only Commons CLI entry from the release LICENSE together with the obsolete 1.1
artifact. Please replace it with a `commons-cli/1.5.0` entry and refresh the
retained Commons CLI NOTICE metadata for the bundled version. The standalone
Apache-2.0 license file does not need to be restored.
##########
README.md:
##########
@@ -31,7 +31,7 @@ achieved through the powerful
[Gremlin](https://tinkerpop.apache.org/gremlin.htm
- **Schema Metadata Management**: VertexLabel, EdgeLabel, PropertyKey, and
IndexLabel
- **Multi-type Indexes**: Exact query, range query, and complex conditions
combination query
-- **Plug-in Backend Store Framework**: Mainly supports `RocksDB`/`HStore` +
`HBase`; other backends available in [legacy
versions](https://hugegraph.apache.org/docs/download/download/) ≤ `1.5.0`
(MySQL/PostgreSQL/Cassandra...)
+- **Plug-in Backend Store Framework**: Since `1.7.0`, supported backends are
`RocksDB`, `HStore`, `HBase`, and `Memory`. Historical versions earlier than
`1.7.0` also supported MySQL, PostgreSQL, Cassandra, ScyllaDB, and Palo.
Review Comment:
🧹 HBase is frozen/deprecated and planned for removal in 2.0, but the public
backend inventories and diagrams still present it identically to maintained
backends. Please annotate this status consistently in the README, server AGENTS
guide, and HBase entry documentation.
##########
install-dist/release-docs/NOTICE:
##########
@@ -1758,28 +1699,6 @@ Components, which can be obtained at:
* https://github.com/Polymer/polymer
-========================================================================
-
-postgres NOTICE
-
-========================================================================
-
-This product contains a modified portion of 'ASM', an open source
Review Comment:
⚠️ This deletion also drops PerfMark’s ASM attribution. The upstream
PerfMark NOTICE lists Catapult, Polymer, and ASM together, and
`perfmark-api-0.25.0.jar` is still bundled. Please restore the ASM subsection
after Polymer and remove only the PostgreSQL-specific attribution.
##########
.serena/project.yml:
##########
@@ -35,12 +35,7 @@ ignore_all_files_in_gitignore: true
# Same syntax as gitignore, so you can use * and **.
# Note: global ignored_paths from serena_config.yml are also applied
additively.
ignored_paths:
- # --- Deprecated backends (focus on RocksDB/HStore only) ---
- - "hugegraph-server/hugegraph-cassandra/**"
- - "hugegraph-server/hugegraph-scylladb/**"
- - "hugegraph-server/hugegraph-mysql/**"
- - "hugegraph-server/hugegraph-postgresql/**"
- - "hugegraph-server/hugegraph-palo/**"
+ # --- HBase is deprecated and planned for removal in 2.0 ---
Review Comment:
🧹 The nearby license-files comment is stale after this cleanup: there are
now 528 tracked license text files, not 585, and legal files must also be
maintained when dependencies are removed or upgraded. Please avoid the volatile
count and state that these files are maintained whenever dependencies change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
