Sure. Will do that. On Sat, Dec 14, 2019 at 8:10 AM Vinoth Chandar <vin...@apache.org> wrote:
> Is this worth documenting for everyone? > > On Fri, Dec 13, 2019 at 4:19 PM Sivabalan <n.siv...@gmail.com> wrote: > > > Thanks. I had some issue w/ local intellij check style configuration. > After > > fixing it, everything is good. > > > > > > On Thu, Dec 12, 2019 at 12:08 PM lamberken <lamber...@163.com> wrote: > > > > > > > > You are welcome. For detail, you can visit HUDI-363, > > > https://issues.apache.org/jira/browse/HUDI-363 > > > > > > > > > > > > best, > > > lamber-ken > > > > > > At 2019-12-13 03:49:12, "Sivabalan" <n.siv...@gmail.com> wrote: > > > >thanks lamber-ken. Sorry, I wasn't paying close attention to these > > > changes. > > > >Don't we make a separate PR (with just the changes pertaining to new > > check > > > >style rules across entire repo) whenever a new change is made to check > > > >style ? I rebased with latest and in order to get my build pass, I > have > > > >already fixed like 20 files and the list keeps growing. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >On Thu, Dec 12, 2019 at 10:14 AM lamberken <lamber...@163.com> wrote: > > > > > > > >> > > > >> > > > >> Hi, @Sivabalan > > > >> > > > >> The new ImportOrder rule split import statements into groups and > > groups > > > >> are separated by one blank line. > > > >> These groups are 1) org.apache.hudi 2) third party imports 3) > > javax > > > >> 4) java 5) static > > > >> > > > >> > > > >> For example > > > >> > > > >> > > > > > > /--------------------------------------------------------------------------- > > > >> package org.apache.hudi.metrics; > > > >> > > > >> import org.apache.hudi.config.HoodieWriteConfig; > > > >> import org.apache.hudi.exception.HoodieException; > > > >> > > > >> import com.google.common.base.Preconditions; > > > >> import org.apache.log4j.LogManager; > > > >> import org.apache.log4j.Logger; > > > >> > > > >> import javax.management.remote.JMXConnectorServer; > > > >> import javax.management.remote.JMXConnectorServerFactory; > > > >> import javax.management.remote.JMXServiceURL; > > > >> > > > >> import java.io.Closeable; > > > >> import java.lang.management.ManagementFactory; > > > >> import java.rmi.registry.LocateRegistry; > > > >> > > > >> public class JmxMetricsReporter extends MetricsReporter { > > > >> > > > >> > > > >> > > > > > > /--------------------------------------------------------------------------- > > > >> > > > >> > > > >> best, > > > >> lamber-ken > > > >> > > > >> 在 2019-12-13 01:01:05,"Sivabalan" <n.siv...@gmail.com> 写道: > > > >> > > > >> Hi folks, > > > >> Is there any recent change wrt checkstyle? Usually I run "mvn > > > package > > > >> -DskipTests" locally to check for any checkstyle and build errors. > And > > > >> travis CI usually stays in line with that. But recently(probably a > > week > > > or > > > >> 10 days), even though my local maven package command succeeds, > travis > > CI > > > >> fails specifically wrt import ordering. > > > >> > > > >> > > > >> When I apply reformat code via intellij, usually I choose just > > "Optimize > > > >> Imports". But this time around, I also tried choosing "Rearrange > > > entries", > > > >> but none helped me in fixing the travis CI failure. > > > >> > > > >> > > > >> Here is my travis CI build: > > > >> > > > > > > https://travis-ci.org/apache/incubator-hudi/jobs/624228722?utm_medium=notification&utm_source=github_status > > > >> > > > >> > > > >> - Do others face this issue or it is just me? > > > >> - Can someone give some pointers on how to go about fixing this? > > > >> > > > >> > > > >> -- > > > >> > > > >> Regards, > > > >> -Sivabalan > > > > > > > > > > > > > > > >-- > > > >Regards, > > > >-Sivabalan > > > > > > > > > -- > > Regards, > > -Sivabalan > > > -- Regards, -Sivabalan