Thanks Owen, I will do that!
Peter
2017. jan. 27. 19:02 ezt írta ("Owen O'Malley" <[email protected]>):
> Just file a jira with the patch that adds the headers.
>
> .. Owen
>
> On Fri, Jan 27, 2017 at 8:08 AM, Peter Vary <[email protected]> wrote:
>
> > Hi,
> >
> > I am working on enabling YETUS pre-commit test so it can help in the
> > reviews.
> >
> > I was running the ASF license check, and I have found the following files
> > are missing the ASF header:
> > common/src/java/org/apache/hadoop/hive/common/
> > classification/RetrySemantics.java
> > druid-handler/src/java/org/apache/hadoop/hive/druid/io/
> > DruidRecordWriter.java
> > jdbc/src/test/org/apache/hive/jdbc/TestHivePreparedStatement.java
> > llap-server/src/java/org/apache/hadoop/hive/llap/io/
> > encoded/LineRrOffsetReader.java
> > llap-server/src/java/org/apache/hadoop/hive/llap/io/
> > encoded/PassThruOffsetReader.java
> > ql/src/test/org/apache/hadoop/hive/ql/parse/TestMergeStatement.java
> > ql/src/test/org/apache/hadoop/hive/ql/plan/TestMapWork.java
> > How do we handle this issue usually?
> > If someone finds it then he will change them, or should we wait for the
> > original authors to add the header?
> >
> > Thanks,
> > Peter
>
