galenwarren commented on pull request #303: URL: https://github.com/apache/flink-statefun/pull/303#issuecomment-1049072113
@austince @tillrohrmann @sjwiesman It just occurred to me that there **is** a circumstance where people might reasonably be implementing the statefun.Context interface -- for unit testing. So this change could break unit tests where statefun.Context is mocked :(. Would that be sufficient reason to leave statefun.Context alone and make this a top-level function? All things being equal, I think it's cleaner as-is, but all things may not be equal. -- 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]
