nwangtw commented on issue #2692: Making emit, ack, and fail thread safe URL: https://github.com/apache/incubator-heron/pull/2692#issuecomment-390739147 @jerrypeng agreed. The concern is that users CAN use the function in the wrong way. Avoiding it in code would be nice; protecting the case might be helpful and acceptable too; otherwise very clear documentation and education is needed (we should have it anyway though). I think maosong prefers to have one of the first 2 options available. Maybe we should have an example code for presave() function if we want to cover stateful process?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services