Fix checkstyle violations (#8585 - Phase 3 - Part 1) #8702
Conversation
There was a problem hiding this comment.
Aren't the license headers too trimmed on this one?
There was a problem hiding this 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.
|
I've just noticed that multiple files line separator changed from LF to CRLF. I'll fix them today. |
sigee
force-pushed
the
fix-checkstyle-violations-part-1
branch
from
c4fb6c6 to
b9203cb
Compare
3 participants
No description provided.