sigee commented on code in PR #8702: URL: https://github.com/apache/storm/pull/8702#discussion_r3286533762
########## storm-client/src/jvm/org/apache/storm/utils/WritableUtils.java: ########## Review Comment: You are right as it is shortened drastically. The reason is that the same license header text was there multiple times (4 times exactly) and I left only a single one. "This file originally comes from the Apache Hadoop project. Changes have been made to the file." If you see the original file in the hadoop project: https://github.com/apache/hadoop/blob/71d216d8c80e7e2682471a2b9819c36a1558033d/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/WritableUtils.java It only contains a short license header. There was no value in the duplication in my opinion. -- 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]
