ShivanshhhG commented on PR #51133: URL: https://github.com/apache/arrow/pull/51133#issuecomment-5820448616
hey, really sorry about that! i've been using AI to help me navigate the C++ codebase since it's my first time contributing here, but i realize i should have engaged more directly with your review instead of just pasting bot responses. to be totally honest, i'm still wrapping my head around the `std::optional` semantics in C++ compared to how it's done in go. i get now that `is_update` needs to be optional because older servers won't send it, so defaulting to false would break backwards compatibility. i'll stop relying on the AI to write my replies. is there anything specific in `server_test.cc` you'd like me to change, or does the recent commit look closer to what you meant? thanks for being patient with me. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
