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

    https://github.com/apache/carbondata/pull/2644#discussion_r214311329
  
    --- Diff: 
hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonTableInputFormat.java
 ---
    @@ -342,60 +341,52 @@ public void refreshSegmentCacheIfRequired(JobContext 
job, CarbonTable carbonTabl
       /**
        * use file list in .carbonindex file to get the split of streaming.
        */
    -  public List<InputSplit> getSplitsOfStreaming(JobContext job, 
AbsoluteTableIdentifier identifier,
    -      List<Segment> streamSegments) throws IOException {
    +  public List<InputSplit> getSplitsOfStreaming(JobContext job, 
List<Segment> streamSegments,
    +      CarbonTable carbonTable, FilterResolverIntf filterResolverIntf) 
throws IOException {
    --- End diff --
    
    You can write an overloaded method for getSplitsOfStreaming. One which 
accepts 3 parameters and one with 4 parameters.
    1.  getSplitsOfStreaming(JobContext job, AbsoluteTableIdentifier 
identifier,List<Segment> streamSegments)
    -- From this method you can the other method and pass null as the 4th 
argument. This will avoid passing null at all places above.
    2. getSplitsOfStreaming(JobContext job, List<Segment> streamSegments, 
CarbonTable carbonTable, FilterResolverIntf filterResolverIntf)


---

Reply via email to