+1 on the ideas. On Tue, Nov 7, 2017 at 6:17 AM, Adam Szita <sz...@cloudera.com> wrote:
> Thanks for all the replies. > > Vihang: Good idea on making everything green before turning this on. For > this purpose I've filed a couple of jiras: > -HIVE-17995 <https://issues.apache.org/jira/browse/HIVE-17995> Run > checkstyle on standalone-metastore module with proper configuration > -HIVE-17996 <https://issues.apache.org/jira/browse/HIVE-17996> Fix ASF > headers > -HIVE-17997 <https://issues.apache.org/jira/browse/HIVE-17997> Add rat > plugin and configuration to standalone metastore pom > > Sahil: there is an umbrella jira (HIVE-13503 > <https://issues.apache.org/jira/browse/HIVE-13503>) for test improvements, > the Yetus integration itself is also a subtask of it. I think any further > improvements on what Yetus features we want to enable should go here too. > > Adam > > > > On 6 November 2017 at 22:02, Sahil Takiar <takiar.sa...@gmail.com> wrote: > > > +1 - think this will be a great addition to Hive. Helping us catch issues > > earlier, keeping the Hive code cleaner, etc. Getting the basic Yetus > checks > > to work seems like a great start, do we have follow JIRAs to get more > YETUS > > tests integrated - e.g. FindBugs? > > > > On Mon, Nov 6, 2017 at 10:29 AM, Vihang Karajgaonkar < > vih...@cloudera.com> > > wrote: > > > > > Thanks Adam for this work. This is definitely useful and a good > addition > > to > > > our test infrastructure. > > > > > > Can we fix the existing issues pointed by Yetus (in a separate JIRA) so > > > that we have a +1 from yetus on the current code? Once that is done, > > > committers can help keep it green as they review patches and merge it. > > > > > > Thanks, > > > Vihang > > > > > > On Mon, Nov 6, 2017 at 9:04 AM, Thejas Nair <thejas.n...@gmail.com> > > wrote: > > > > > > > +1 > > > > Yes, I think this can help us catch many issues early on, it will be > > very > > > > useful! > > > > > > > > > > > > On Mon, Nov 6, 2017 at 7:43 AM, Adam Szita <sz...@cloudera.com> > wrote: > > > > > > > > > Hi all, > > > > > > > > > > As a next step of test subsystem improvements we would like to have > > the > > > > > Yetus check integrated with the ptest framework. This means that > > > > whenever a > > > > > new patch is uploaded - along with the already existing Precommit > > test > > > > run > > > > > - Hive's Yetus patch check script would be triggered also. This > > script > > > > runs > > > > > checkstyle, findbugs, ASF license check, etc with and without the > > > > submitted > > > > > patch applied and reports the diffs (i.e. how many checkstyle > > problems > > > > does > > > > > the patch introduce). > > > > > > > > > > It would be executed parallel to the ptest test execution and > report > > > back > > > > > the results as a (another) jira comment to the issue in question. > > > > > In the last days I've been working on this (HIVE-16748) and a patch > > is > > > > > ready to make this happen. A sample Yetus result comment is > available > > > at > > > > > https://issues.apache.org/jira/browse/HIVE-16748? > > > > > focusedCommentId=16218616&page=com.atlassian.jira. > > > > > plugin.system.issuetabpanels:comment-tabpanel#comment-16218616 > > > > > > > > > > We think this would be a useful tool for us developers and would > like > > > to > > > > go > > > > > ahead with this change, but we're also curious about your input in > > this > > > > > matter. Please let us know what you think about this change. > > > > > > > > > > Thanks, > > > > > Adam > > > > > > > > > > > > > > > > > > > > -- > > Sahil Takiar > > Software Engineer at Cloudera > > takiar.sa...@gmail.com | (510) 673-0309 > > >