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
---