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

    https://github.com/apache/carbondata/pull/1477#discussion_r149944113
  
    --- Diff: 
integration/presto/src/main/java/org/apache/carbondata/presto/CarbondataPageSource.java
 ---
    @@ -78,10 +78,6 @@ private CarbondataPageSource(List<Type> types, 
RecordCursor cursor) {
         this.readers = createStreamReaders();
       }
     
    -  @Override public long getTotalBytes() {
    --- End diff --
    
    The Presto 0.186 has remove this method from specification, so we have to 
remove it from our code as well


---

Reply via email to