Summary: This is a pretty obvious thing to do. There's not really any reason to have the data and metadata separately since we don't need to reallocate.
Reviewed By: jbeshay
Differential Revision: D36237370
fbshipit-source-id: 093ad7fb2c54b596ea5cc327ffcc24de1748d362
Summary: Currently writePacketsGroup returns the number of packets it sent. Update it to return the number of bytes has been sent too.
Reviewed By: mjoras
Differential Revision: D34280604
fbshipit-source-id: 5cc4047401a00054a2bc0a0e7429a3beaa046f44
Summary: This makes the RequestGroup its own struct so we don't have so much redundant information in the requests.
Reviewed By: shodoco
Differential Revision: D32175496
fbshipit-source-id: 7365f55133f2eeebd16b8a534e7e4ce08842d327
Summary: This doesn't belong in the generic state. Untangling it is a little difficult, but I think this solution is cleaner than having it in the generic state.
Reviewed By: JunqiWang
Differential Revision: D29856391
fbshipit-source-id: 1042109ed29cd1d20d139e08548d187b469c8398
