Github user jackylk commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2816#discussion_r227680646
  
    --- Diff: store/CSDK/CarbonReader.cpp ---
    @@ -17,6 +17,7 @@
     
     #include "CarbonReader.h"
     #include <jni.h>
    +#include <sys/time.h>
    --- End diff --
    
    I think system header file should be before carbon header file, so 
"CarbonReader.h" should be moved down


---

Reply via email to