test: Add regression tests for protobuf >= 7.34.0 compatibility (#6435) by Jwrede · Pull Request #6450 · feast-dev/feast
Jwrede
changed the title
test: add regression tests for protobuf >= 7.34.0 compatibility (#6435)
test: Add regression tests for protobuf >= 7.34.0 compatibility (#6435)
…t-dev#6435) PR feast-dev#6015 replaced MessageToDict with a custom dict builder in feature_server.py, which incidentally fixed the crash caused by the removed float_precision kwarg in protobuf >= 7.34.0. Add explicit regression tests to prevent reintroduction: - Verify convert_response_to_dict never calls MessageToDict - Verify double precision round-trips without float_precision kwarg Fixes feast-dev#6435 Assisted-by: Claude Opus 4.6 Signed-off-by: Jonathan Wrede <wrede.jonathan00@gmail.com>
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