MisterRaindrop opened a new pull request, #796:
URL: https://github.com/apache/iceberg-cpp/pull/796
Adds the first `iceberg_hive` code that talks to a Hive Metastore over
Thrift, plus CI coverage.
- **`ParseHmsUris()`** — tolerant HMS URI parser: `thrift://host:port`,
bare `host:port`, default port 9083, and comma-separated HA lists.
Rejects empty hosts and invalid ports.
- **`HmsClient::Connect()`** — sets up the `TSocket → TTransport →
TBinaryProtocol → ThriftHiveMetastoreClient` chain (transport and
timeouts from `HiveCatalogProperties`), translating Thrift exceptions
to `kIOError`. Thrift types stay out of the public header via a pImpl.
- **CI** — new `Ubuntu Hive` job builds with `-DICEBERG_BUILD_HIVE=ON`
(bundled Thrift) and runs `hive_catalog_test`.
Tests: `hms_client_test.cc` adds 16 cases; CI reports `18/18 passed`.
Part of the HiveCatalog port.
--
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]
