+1 on removing the import control. The original idea to include the checkstyle.xml is to enforce some coding style guidelines, not to strictly control the imports. W/ the outdated import control list, it practically does not serve the purpose...
On Mon, Jul 11, 2016 at 4:02 PM, Navina Ramesh <nram...@linkedin.com.invalid > wrote: > Hi Samza devs, > > Lately, with the major re-works such as standalone, multithreading etc, it > is getting harder to keep track of the package/class dependencies in > import-control.xml. > > What I have noticed is that we don't bother removing the class/package > dependencies when it is no longer valid. This is not raised as a fault by > the checkstyle plugin. However, this means that import-control.xml is going > to continuously grow without adding much value. > > Does anyone find this particularly useful? Should we consider removing the > import-control verification from checkstyle? > > Thanks! > -- > Navina R. >
