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

    https://github.com/apache/carbondata/pull/2816#discussion_r228025429
  
    --- Diff: store/CSDK/CMakeLists.txt ---
    @@ -1,17 +1,17 @@
    -cmake_minimum_required (VERSION 2.8)
    -project (CJDK)
    +cmake_minimum_required(VERSION 2.8)
    --- End diff --
    
    I think no need to add license header in this file.
    CMakeLists.txt like pom.xml, it's not code file. 
    Tensorflow and caffe project also didn't add license header in 
CMakeLists.txt .


---

Reply via email to