icexelloss commented on PR #34392: URL: https://github.com/apache/arrow/pull/34392#issuecomment-1511711206
> Detecting is not good enough. GetCurrentTime() needs a valid time when called, so there needs to be code that ensures this. The MemoStore constructor can't do this because at that time there is no input yet. If the `GetCurrenTime() returns something that the caller can detect as invalid (e.g., negative value), then the caller can treat it as if there is no data and skip the current processing? -- 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]
