freezhan edited a comment on pull request #12157: URL: https://github.com/apache/flink/pull/12157#issuecomment-656483534
In addition, our internal temporary repair method is ```java this.prefix = prefix.substring(0, Math.min(prefix.length(), 200)) + "-" + EncodingUtils.hex(EncodingUtils.md5(prefix)); ``` how about that? Can add this sentence before calling the `TransactionalIdsGenerator ` constructor, and then add a length check inside the constructor. But I still think it makes sense to let users pass it in. If the user does not specify the prefix parameter, the default is a whole bunch of context characters(Keep it the same as before) ---------------------------------------------------------------- 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: us...@infra.apache.org
