Rob Howley

would it perhaps be clearer to just go full async context manager? somewhat more explicit that this isnt going to impact the sync code paths?

@franciscojavierarceo just a heads up, [the remaining integration test failure](https://github.com/feast-dev/feast/actions/runs/11488391804/job/31975025552?pr=4642#step:15:907) comes from [here](https://github.com/feast-dev/feast/blob/master/sdk/python/tests/integration/online_store/test_python_feature_server.py#L61). the push works, but the subsequent retrieval to confirm it's there, in the case of dynamo, is...