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