Repository navigation
refactor(proto): remove unused codec scaffolding - #159
Merged
Merged
Conversation
Replace Consul creation macros with typed bases, a private owning factory and explicit DLL entry points. Preserve interface registration, disabled updates and request behavior, and add construction/lifecycle checks with bilingual migration guidance. BREAKING CHANGE: Consul implementation and plugin class layouts change. Rebuild Consul and its consumers together; all existing public types and logical header paths remain available.
Read hash blocks and write digest bytes without alignment or aliasing assumptions. Use C++20 rotations and provide explicit legacy MSVC methods for persisted-key migration. BREAKING CHANGE: default MurmurHash3 now uses canonical little-endian blocks and output. Old MSVC IDs, shard mappings and persisted keys require an explicit migration or the matching legacy MSVC method.
Bumps [fast-uri](https://github.com/fastify/fast-uri) from 3.1.8 to 3.1.9. - [Release notes](https://github.com/fastify/fast-uri/releases) - [Commits](fastify/fast-uri@v3.1.8...v3.1.9) --- updated-dependencies: - dependency-name: fast-uri dependency-version: 3.1.9 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
# Conflicts: # docs/reference/utility.md # docs/reference/utility_CN.md # src/core/hash/murmur_hash.hpp # tests/core/README.md # tests/core/README_CN.md
Check a successful live-listener negative control, retain the exercised TCP peer across manager shutdown, and require a remote EOF or reset before stopping that peer. Check listener closure using an exact refused new connection instead of global immediate port rebinding. Keep existing endpoint ownership, manager state, main-thread callbacks and pending HTTP cancellation assertions. The original macOS CI bind conflict holder was not reproduced locally; no production shutdown or reuse-address behavior changes.
Use a private io_context, async_connect and the existing five-second deadline. Cancel and close the probe socket on expiry, return typed timed_out rather than operation_aborted, and drain connect and timer callbacks before returning. Keep all live-listener, retained-peer and owner-state assertions and the existing test timeout. Production shutdown remains unchanged.
Make `<ark/hash/murmur_hash.hpp>` usable with the C++20 standard library alone by removing platform/macro headers and replacing legacy constant macros with unsigned literals. The consistent-hash consumer now includes its own formatting dependency explicitly. Hash values and public signatures are unchanged from the preceding portability fix. A permanent isolated-header target compiles all seven methods without framework or third-party include paths and runs through `check-core`. English/Chinese guides distinguish header independence from the existing `ark::core` package dependencies. Local Debug and AddressSanitizer + UndefinedBehaviorSanitizer acceptance passed for the unchanged Hash implementation (62 core cases /69,857 assertions; leak detection disabled). The combination at `5893a0c2beaaf3a0b70acd8107775534fd8cbb94` also passed `check-core`, the independent header check and clang-format 18. It includes the preceding Net shutdown test repair; actual Net DSO and stronger peer/listener checks passed separately and in the complete local combination. Fresh documentation/CI policy checks and the protected cross-platform candidate remain required. No compile-time performance claim. The current head 892ccba includes preceding queue candidate 62a656b. Native implementation, tests and CMake inputs are byte-identical to the previously tested local revision 5893a0c; only the already validated documentation lockfile can differ. Fresh documentation and CI policy checks passed at the current head. Final cross-platform acceptance remains subject to its own protected merge candidate. --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
nick2781
changed the base branch from
refactor/hash-header-dependencies
to
main
October 26, 2026 05:43
nick2781
marked this pull request as ready for review
October 26, 2026 05:43
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove the unused internal codec scaffold whose encode/decode methods accepted no payload and returned success unconditionally. The three files have no tracked external callers, build inputs, public header mapping or installation rule. Existing schemas, generated Protobuf messages and active Bus serialization are preserved.
English/Chinese Protobuf and directory guides describe the actual
SerializeToString/ParseFromStringpath and its failure behavior.Local native acceptance covers 70 core/Protobuf cases /69,951 assertions and the preceding Net DSO shutdown test repair; the full local combination containing this change also passed actual Net DSO, Etcd DSO and independent Hash header checks. The original Bus configuration regressions passed 8 cases /169 assertions. At
dd02b81cef484fd4389fe11ff4e177f1734f6eac, 155 CI policy tests, Nimbus (137 pages /275 references), actual bilingual HTML articles/Markdown exports and clang-format 18 passed. Protobuf, Abseil and utf8_range use one installation prefix. Hosted matrix acceptance and publication remain pending.The current head 15cc26b includes preceding queue candidate 0ed2458. Native implementation, tests and CMake inputs are byte-identical to the previously tested local revision dd02b81; only the already validated documentation lockfile can differ. Fresh documentation and CI policy checks passed at the current head. Final cross-platform acceptance remains subject to its own protected merge candidate.