Misc optimize

[johnlanni]

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

---

Note

Introduce xxhash-based cached protobuf Message hashing and use it across Envoy (listener/maps/config hashing), plus HIGRESS-specific fixes/features including WASM VM proactive rebuild, end_stream handling, new stats, and minor HTTP/redirect tweaks.

• Protobuf/Build:
  • Add xxHash dependency and patch google::protobuf::Message with cached ComputeHashValue()/GetCachedHashValue().
  • Expose HashCachedMessageUtil and switch various hash/equality maps to it (e.g., FilterChainMatch, FilterChain, listener default FC comparison).
  • Use cached hashing for listener/config hashes (e.g., ListenerManagerImpl, RDS RouteConfig hashing).
• HIGRESS/WASM:
  • Add WASM VM rebuild flow (PluginHandleSharedPtrThreadLocal::rebuild), new rebuild_total stat, lower recover interval, and filter-state trigger (wasm_need_rebuild).
  • Expose plugin_vm_memory property; tests for VM memory and proactive rebuild.
• HTTP/Conn Manager/Filters:
  • Rename ALIMESH to HIGRESS in stats macros; add keep-alive header handling.
  • Ensure end_stream consistency when injecting data (set remote_decode_complete_/local_complete_).
  • Preserve downstream remote address on internal redirect recreation.
  • Header access fallback via StreamInfo when request headers pointer is null.
• Custom Response Redirect:
  • Preserve original upstream cluster info and response code during internal redirects; set response code in StreamInfo.
• Config/Stats:
  • Add GAUGE(last_update_success) to subscription stats; set in gRPC subscription (guarded by macro).
• Tests:
  • Extensive tests for cached hashing (struct/list/any/nested), listener/protobuf size updates, WASM VM memory and rebuild scenarios.

^{Written by Cursor Bugbot for commit 45d3e7f. This will update automatically on new commits. Configure here.}

[lingma-agents]

CodeReview流程已终止

[cursor]

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

[cursor]

Bug: Hash Functor Returns Wrong Type, Catastrophic Collisions

The hash functor operator() in HashCachedMessageUtil returns bool instead of std::size_t. When used as the hash template parameter in absl::node_hash_map and absl::flat_hash_map, this causes the hash value to be implicitly converted to bool (0 or 1), resulting in catastrophic hash collisions where all messages hash to either 0 or 1, breaking hash map functionality.

 

[cursor]

Bug: Silent Wasm Rebuild Failure

The filter state key "wasm_rebuild" doesn't match the constant WasmRebuildKey defined as "wasm_need_rebuild" in source/extensions/common/wasm/context.cc. This mismatch prevents the rebuild flag from being detected, causing the rebuild feature to silently fail in tests and production.