[GitHub] [incubator-hudi] taherk77 commented on issue #849: [HUDI-62] Added index handle times

2019-08-28 Thread GitBox
taherk77 commented on issue #849: [HUDI-62] Added index handle times
URL: https://github.com/apache/incubator-hudi/pull/849#issuecomment-525739928
 
 
   > @taherk77 you can click on "Details" and it will take you to the job 
page.. there you can click again on the failing job and look at the logs
   > 
   > ```
   >  [WARN] 
/home/travis/build/apache/incubator-hudi/hudi-client/src/main/java/org/apache/hudi/HoodieWriteClient.java:175:25:
 WhitespaceAround: '=' is not preceded with whitespace. [WhitespaceAround]
   > [WARN] 
/home/travis/build/apache/incubator-hudi/hudi-client/src/main/java/org/apache/hudi/HoodieWriteClient.java:175:26:
 WhitespaceAround: '=' is not followed by whitespace. Empty blocks may only be 
represented as {} when not part of a multi-block statement (4.1.3) 
[WhitespaceAround]
   > [WARN] 
/home/travis/build/apache/incubator-hudi/hudi-client/src/main/java/org/apache/hudi/HoodieWriteClient.java:177:
 Line is longer than 120 characters (found 133). [LineLength]
   > ```
   > 
   > Looks like you have checkstyle failures.. Typically, people first build, 
run the tests locally , get them to pass and then push it to gh. may be we can 
try that?
   
   Hey Sorry for that @vinothchandar, I did run all the junits before pushing 
the code.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-hudi] taherk77 commented on issue #849: [HUDI-62] Added index handle times

2019-08-27 Thread GitBox
taherk77 commented on issue #849: [HUDI-62] Added index handle times
URL: https://github.com/apache/incubator-hudi/pull/849#issuecomment-525294717
 
 
   @vinothchandar Hi Please can you tell me why Travis is failing? Sorry never 
worked with Travis so not aware what needs to be done here. I see warnings for 
spacing? Is there a specific format I am supposed to use?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services