* move bufio reader creation out of for loop
if the bufio reader is created in the for loop we get unmarshaling errors
* fix linter issue
* add fixed ut
* fix existing unit test flake due to closing pipe on error
a previous fix ensured the socket closed on error, but this caused an existing ut to nondeterministically fail
without the previous fix, the socket wouldn't have been closed on error
* make read inline
* added lockedfileapi support for CNI
* fixed interface changes
* addressed comments
fixed ut
* addressed comments
* fixed copy to buffer part in writer api
* fixed copy to buffer part in writer api
* keeping old code not changing it.
* Refactor UTs to cleanup UTs and increase UT coverages
* User assert for consistency
* Applied comments and resolve lint error
* Delete unnecessary license header
Co-authored-by: JungukCho <jungukcho@microsoft.com>
QxBytes
Paul Yu
tamilmani1989