vaijosh opened a new pull request, #3041:
URL: https://github.com/apache/hugegraph/pull/3041
### Description
While investigating issue #3030, the underlying bug could not be directly
reproduced on the current master branch. However, during the investigation, I
identified several improvements for docker/hbase/README.md that will be useful
for verifying HBase-related issues in the future.
Specifically, this PR splits the local verification process into two clear
paths (Standalone vs. Fully Containerized). It also enhances the HBase Docker
artifacts by making hbase.master.hostname and hbase.regionserver.hostname fully
configurable via environment variables.
---
### Main Changes
#### 1. Documentation Updates (`docker/hbase/README.md`)
* **Dual-Path Quick Start:** Restructured the guide into two explicit
validation paths:
* **Option 1:** Standalone HugeGraph engine backed by Dockerized HBase.
* **Option 2:** Fully containerized setup (Docker HugeGraph + Docker
HBase) using a shared network.
* **Automated Setup Snippets:** Added programmatic `perl` scripts to
dynamically swap backend/serializer properties (`rocksdb` $\rightarrow$
`hbase`) and rewrite host configurations on the fly.
* **REST Verification Runbook:** Checked in an explicit, end-to-end
verification script utilizing the native `/traversers/` REST APIs (`kout`,
`rays`) to securely validate graph space persistence.
* **Troubleshooting Reference:** Added a scannable table mapping common
system errors (like `UnknownHostException` or port conflicts) directly to their
respective resolutions.
#### 2. Docker Environments (`docker/hbase/`)
* **Flexible Hostnames (`docker-compose.hbase.yml`):** Replaced hardcoded
hostnames with a configurable environment variable fallback
(`${HBASE_HOSTNAME:-hbase}`).
* **Dynamic Property Injection (`entrypoint.sh`):** Added a
`set_xml_property_value` helper function inside the entrypoint sequence. This
cleanly updates `hbase.master.hostname` and `hbase.regionserver.hostname`
configurations inside `hbase-site.xml` at runtime based on the container
environment.
---
### Verifying of the issue
Verified the issue as follows
* Verified issue **#3030** using the updated setup runbook to ensure the
environment is correctly configured.
* Followed **Option 1** by building the project locally (`mvn clean
package -DskipTests`), initializing the store backend (`init-store.sh`), and
running the sequential REST traversal scripts.
* Followed **Option 2** by building the `hugegraph/server:dev` target
image and executing the one-shot store initializations inside an isolated
container network.
* Confirmed that the `hugegraph-server` logs were entirely clean of the
targeted warnings during operation; specifically, **no** instances of the
following log patterns were observed:
> `[WARN] o.a.h.t.ServerInfoManager - ServerInfo is missing:
DEFAULT-aikg/server-1`
> `[WARN] o.a.h.t.ServerInfoManager - ServerInfo is missing:
DEFAULT-aikg/server-1, may be cleared before`
### Does this PR potentially affect the following parts?
- [ ] Dependencies
- [x] Modify configurations *(Updates localized Docker helper configs and
verification scripts)*
- [ ] The public API
- [ ] Other affects
- [ ] Nope
### Documentation Status
- [x] `Doc - Done` *(The primary objective of this PR is introducing the
HBase validation runbook docs)*
--
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]
