async deserialization functions are unintuitive and possibly insecure #3977
Description
Describe the bug
Discussion has highlighted that turning the logical_plan_from_bytes() method async was probably not a great idea. We should revert it, but not break the progress being made on deltalake tables in Ballista.
To Reproduce
No reproduction steps, but it doesn't seem like a feature either.
Expected behavior
The trait should not be async.