You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by dhatchayani <gi...@git.apache.org> on 2018/04/19 07:49:55 UTC

[GitHub] carbondata pull request #2187: [CARBONDATA-2362] Changing the Cacheable obje...

GitHub user dhatchayani opened a pull request:

    https://github.com/apache/carbondata/pull/2187

    [CARBONDATA-2362] Changing the Cacheable object from DataMap to Wrapper

    (1) Cacheable object is changed from dataMap to wrapper
    (2) Blocklet info is stored as binary to dataMap
    
     - [ ] Any interfaces changed?
     
     - [ ] Any backward compatibility impacted?
     
     - [ ] Document update required?
    
     - [x] Testing done
            Manual Testing
           
     - [ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA. 
    


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dhatchayani/carbondata optimizations_cache

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/carbondata/pull/2187.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2187
    
----
commit 889b90c6e99762f55fb42a516733bc38590f4cfd
Author: dhatchayani <dh...@...>
Date:   2018-04-19T07:18:15Z

    [CARBONDATA-2362] Changing the Cacheable object from DataMap to Wrapper

----


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4232/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    retest this please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4180/



---

[GitHub] carbondata pull request #2187: [CARBONDATA-2362] Changing the Cacheable obje...

Posted by manishgupta88 <gi...@git.apache.org>.
Github user manishgupta88 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2187#discussion_r184278866
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/datastore/DataRefNode.java ---
    @@ -141,5 +141,5 @@ DimensionRawColumnChunk getDimensionChunk(FileHolder fileReader, int blockIndexe
        * @param pageNumber
        * @return
        */
    -  int getPageRowCount(int pageNumber);
    +  int getPageRowCount(int pageNumber) throws IOException;
    --- End diff --
    
    Avoid throwing IOException from this interface methods. Instead catch the IO exception in the caller methods and wrap it in RuntimeException and throw


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5233/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4225/



---

[GitHub] carbondata pull request #2187: [CARBONDATA-2362] Changing the Cacheable obje...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2187#discussion_r182936215
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletDataMap.java ---
    @@ -978,15 +968,6 @@ public long getMemorySize() {
         return memoryUsed;
       }
     
    -  public TableBlockIndexUniqueIdentifier getTableBlockUniqueIdentifier() {
    --- End diff --
    
    Now is it required this class to implement Cacheable? 


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/4509/



---

[GitHub] carbondata pull request #2187: [CARBONDATA-2362] Changing the Cacheable obje...

Posted by manishgupta88 <gi...@git.apache.org>.
Github user manishgupta88 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2187#discussion_r184281016
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletDataMapFactory.java ---
    @@ -121,7 +123,7 @@ public DataMapWriter createWriter(Segment segment) {
             segmentMap.get(segment.getSegmentNo());
         if (tableBlockIndexUniqueIdentifiers == null) {
           tableBlockIndexUniqueIdentifiers =
    -          BlockletDataMapUtil.getTableBlockUniqueIdentifiers(segment, identifier.getTablePath());
    +           BlockletDataMapUtil.getTableBlockUniqueIdentifiers(segment, identifier.getTablePath());
    --- End diff --
    
    check for proper identation for this line


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Retest this please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/4529/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/4550/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by manishgupta88 <gi...@git.apache.org>.
Github user manishgupta88 commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    LGTM


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    retest this please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5370/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4156/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Retest this please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    retest this please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/4534/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/4520/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4044/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    retest sdv please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4253/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5424/



---

[GitHub] carbondata pull request #2187: [CARBONDATA-2362] Changing the Cacheable obje...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2187#discussion_r182937982
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/BlockletDetailInfo.java ---
    @@ -83,6 +84,22 @@ public void setVersionNumber(short versionNumber) {
       }
     
       public BlockletInfo getBlockletInfo() {
    +    if (null == blockletInfo && null != blockletInfoBinary && blockletInfoBinary.length > 0) {
    +      blockletInfo = new BlockletInfo();
    +      ByteArrayInputStream stream = new ByteArrayInputStream(blockletInfoBinary);
    +      DataInputStream inputStream = new DataInputStream(stream);
    +      try {
    +        blockletInfo.readFields(inputStream);
    +      } catch (IOException e) {
    +        e.printStackTrace();
    +      } finally {
    +        try {
    +          inputStream.close();
    +        } catch (IOException e) {
    +          e.printStackTrace();
    --- End diff --
    
    remove it and throw exception


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Retest sdv please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4127/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4168/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5413/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/4519/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    retest this please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5307/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4214/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    retest this please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5381/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5406/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/4552/



---

[GitHub] carbondata pull request #2187: [CARBONDATA-2362] Changing the Cacheable obje...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2187#discussion_r182937969
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/BlockletDetailInfo.java ---
    @@ -83,6 +84,22 @@ public void setVersionNumber(short versionNumber) {
       }
     
       public BlockletInfo getBlockletInfo() {
    +    if (null == blockletInfo && null != blockletInfoBinary && blockletInfoBinary.length > 0) {
    +      blockletInfo = new BlockletInfo();
    +      ByteArrayInputStream stream = new ByteArrayInputStream(blockletInfoBinary);
    +      DataInputStream inputStream = new DataInputStream(stream);
    +      try {
    +        blockletInfo.readFields(inputStream);
    +      } catch (IOException e) {
    +        e.printStackTrace();
    --- End diff --
    
    remove it and throw exception


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4198/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Retest this please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4194/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5324/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Retest this please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4187/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4246/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/4505/



---

[GitHub] carbondata pull request #2187: [CARBONDATA-2362] Changing the Cacheable obje...

Posted by manishgupta88 <gi...@git.apache.org>.
Github user manishgupta88 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2187#discussion_r184281751
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/table/TableInfo.java ---
    @@ -97,6 +102,14 @@ public TableSchema getFactTable() {
       public void setFactTable(TableSchema factTable) {
         this.factTable = factTable;
         updateParentRelationIdentifier();
    +    updateIsSchemaModified();
    +  }
    +
    +  private void updateIsSchemaModified() {
    +    if (null != factTable.getSchemaEvalution()) {
    +      isSchemaModified =
    --- End diff --
    
    Add a comment here as below
    If schema evolution entry list size  is > 1 that means n alter operation is performed which has added the new schema entry in the schema evolution list. Currently apart from create table schema evolution entries are getting added only in the alter operations.


---

[GitHub] carbondata pull request #2187: [CARBONDATA-2362] Changing the Cacheable obje...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2187#discussion_r182937925
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/BlockletDetailInfo.java ---
    @@ -50,6 +49,8 @@
     
       private BlockletInfo blockletInfo;
    --- End diff --
    
    I think it should be transient


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4022/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    retest this please


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5395/



---

[GitHub] carbondata pull request #2187: [CARBONDATA-2362] Changing the Cacheable obje...

Posted by manishgupta88 <gi...@git.apache.org>.
Github user manishgupta88 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2187#discussion_r184280492
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/UnsafeMemoryDMStore.java ---
    @@ -101,6 +102,7 @@ public void addIndexRow(DataMapRow indexRow) throws MemoryException {
       }
     
       private void addToUnsafe(CarbonRowSchema schema, DataMapRow row, int index) {
    +    schema.setDataType(DataTypeUtil.valueOf(schema.getDataType(), 0, 0));
    --- End diff --
    
    Remove this logic from here and add a method in AbstractMemoryDMStore called setSchemaDataType and set the datatypes at once in that method


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5216/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4141/



---

[GitHub] carbondata pull request #2187: [CARBONDATA-2362] Changing the Cacheable obje...

Posted by dhatchayani <gi...@git.apache.org>.
Github user dhatchayani closed the pull request at:

    https://github.com/apache/carbondata/pull/2187


---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4239/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5357/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5398/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5392/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/5337/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by CarbonDataQA <gi...@git.apache.org>.
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    Build Failed with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4228/



---

[GitHub] carbondata issue #2187: [CARBONDATA-2362] Changing the Cacheable object from...

Posted by ravipesala <gi...@git.apache.org>.
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/2187
  
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/4531/



---