Hi, The patch is waiting for review, commit. Could someone take a look at it? https://issues.apache.org/jira/browse/HIVE-15749 <https://issues.apache.org/jira/browse/HIVE-15749>
Thanks, Peter > On Jan 27, 2017, at 9:28 PM, Peter Vary <[email protected]> wrote: > > Thanks Owen, I will do that! > > Peter > > 2017. jan. 27. 19:02 ezt írta ("Owen O'Malley" <[email protected] > <mailto:[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] > <mailto:[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
