fix: Use After Free in PacketReader

[discord9]

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

related issue: #66
Refactor packet_reader.rs to avoid Use After Free due to dropping Vec while still holding packets of u8 slice from it.

• impl NomBytes type which allow nom to parse bytes::Bytes which is smart pointer to array of u8, hence avoid uaf
• reuse buffer whenever possible to improve performance
• make sure packet still have a lifetime arg, so to prevent breaking change to api
• correct both impl of next() and next_async() to prevent UAF or unsafe code
• add tests that run to those large packet dealing logics(Which disclosure another bug about having packet size of exactly U24_MAX would also cause nom to refuse parse packets due to lack of opt, fixed)
• cargo test is passed

[discord9]

@PsiACE might want to take a look?

[discord9]

The style warning is all in clickhouse/*, I'm not sure if I should fix it in case might break something,
The MacOS CI failure seems to be related to wrong CI setting in Mac env?

[PsiACE]

Thanks @discord9 . I will check tonight, I am now a community contributor.