[ 
https://issues.apache.org/jira/browse/HBASE-24973?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

yuqi updated HBASE-24973:
-------------------------
    Description: 
Currently, read point parameter in method StoreFlush#performFlush is useless 
and can be safely removed.
and then method StoreFlush#
See below 


{code:java}
// Some comments here
  /**
   * Performs memstore flush, writing data from scanner into sink.
   * @param scanner Scanner to get data from.
   * @param sink Sink to write data to. Could be StoreFile.Writer.
   * @param smallestReadPoint Smallest read point used for the flush.
   * @param throughputController A controller to avoid flush too fast
   */
  protected void performFlush(InternalScanner scanner, CellSink sink,
      long smallestReadPoint, ThroughputController throughputController) throws 
IOException
{code}



  was:
Currently, read point parameter in method StoreFlush#performFlush is useless 
and can be safely removed.
See below 


{code:java}
// Some comments here
  /**
   * Performs memstore flush, writing data from scanner into sink.
   * @param scanner Scanner to get data from.
   * @param sink Sink to write data to. Could be StoreFile.Writer.
   * @param smallestReadPoint Smallest read point used for the flush.
   * @param throughputController A controller to avoid flush too fast
   */
  protected void performFlush(InternalScanner scanner, CellSink sink,
      long smallestReadPoint, ThroughputController throughputController) throws 
IOException
{code}




> Remove read point parameter in method StoreFlush#performFlush and 
> StoreFlush#createScanner
> ------------------------------------------------------------------------------------------
>
>                 Key: HBASE-24973
>                 URL: https://issues.apache.org/jira/browse/HBASE-24973
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: yuqi
>            Priority: Minor
>
> Currently, read point parameter in method StoreFlush#performFlush is useless 
> and can be safely removed.
> and then method StoreFlush#
> See below 
> {code:java}
> // Some comments here
>   /**
>    * Performs memstore flush, writing data from scanner into sink.
>    * @param scanner Scanner to get data from.
>    * @param sink Sink to write data to. Could be StoreFile.Writer.
>    * @param smallestReadPoint Smallest read point used for the flush.
>    * @param throughputController A controller to avoid flush too fast
>    */
>   protected void performFlush(InternalScanner scanner, CellSink sink,
>       long smallestReadPoint, ThroughputController throughputController) 
> throws IOException
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to