OK, I will remove it in my RAT cleanup patch once I make sure the removal passes when I do the rest of the RAT clean, too:
https://gerrit.cloudera.org/#/c/4361/ That will take a few hours for the test to complete. On Mon, Sep 12, 2016 at 4:06 PM, Alex Behm <[email protected]> wrote: > Let's remove it then! > > On Mon, Sep 12, 2016 at 3:24 PM, Jim Apple <[email protected]> wrote: > > > The test is complete; removing that directory did not cause any tests to > > fail. > > > > On Mon, Sep 12, 2016 at 11:46 AM, Jim Apple <[email protected]> > wrote: > > > > > I'm running a test now to see if we can just delete that whole > directory. > > > Full tests take a few hours, so I'll post when I have more info. > > > > > > 'mstr' doesn't seem to appear as a meaningful string in our repo, based > > on > > > a cursory grep. > > > > > > On Mon, Sep 12, 2016 at 10:53 AM, Todd Lipcon <[email protected]> > wrote: > > > > > >> +Carl Steinbach since it seems he was the original author (and not > sure > > if > > >> he pays attention to dev@) > > >> > > >> On Mon, Sep 12, 2016 at 10:49 AM, Jim Apple <[email protected]> > > wrote: > > >> > > >> > In order to do a 2.7.0 release, we must ensure that everything in > the > > >> > release is OK with http://www.apache.org/legal/resolved.html > > >> > > > >> > I am using Apache RAT to do that. It is flagging the files in > > >> > testdata/data/mstr. Those files do not have a very informative git > log > > >> > history: > > >> > > > >> > https://github.com/apache/incubator-impala/commits/ > > >> > master/testdata/data/mstr > > >> > > > >> > Can anyone vouch for where they came from? This is a 2.7.0 release > > >> blocker. > > >> > > > >> > Thanks, > > >> > Jim > > >> > > > >> > > >> > > >> > > >> -- > > >> Todd Lipcon > > >> Software Engineer, Cloudera > > >> > > > > > > > > >
