asdf2014 commented on a change in pull request #1172:
URL:
https://github.com/apache/incubator-seatunnel/pull/1172#discussion_r793220432
##########
File path:
seatunnel-common/src/main/java/org/apache/seatunnel/common/utils/StringTemplate.java
##########
@@ -37,7 +37,7 @@ public static String substitute(String str, String
timeFormat) {
final SimpleDateFormat sdf = new SimpleDateFormat(timeFormat);
final String formattedDate = sdf.format(new Date());
- final Map<String, String> valuesMap = new HashMap(5);
+ final Map<String, String> valuesMap = new HashMap(3);
Review comment:
Hi @leo65535 @CalvinKirs, thanks for your comment, I agree with you, but
if we use 5, it will resize to 8, which is unsuitable. If we use 3, it will
resize to 4, which is quit enough here. So, I believe this improvement is
necessary, right
--
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]
