igalshilman commented on pull request #114: URL: https://github.com/apache/flink-statefun/pull/114#issuecomment-631670579
Hi @slinkydeveloper, I've omitted that commit since it included additional changes beside the test. (Docker files, print statements, etc') I've took the test code and refactored it a little and now it is included. All and all, I think that the PR is in a really good shape, I've validated it locally. @tzulitai when merging, feel free to squash everything to a single commit / re-arrange them a little bit, since some of the changes are already bundled anyways. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
