wenjin272 commented on issue #907: URL: https://github.com/apache/flink-agents/issues/907#issuecomment-4976588853
Thanks for reporting this. I confirmed the issue on the current main branch. Since the Java implementation applies `String.replace` sequentially, placeholders introduced by an earlier substitution may be expanded by a later one, making the result dependent on the map iteration order. The native Python formatter performs a single-pass substitution and is not affected. I think that the Java implementation should only scan placeholders from the original template, with regression coverage for both `formatString` and `formatMessages`. Thanks for indicating that you’re willing to submit a PR, I’ve assigned the issue to you. Please let us know if you have any questions along the way. -- 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]
