[jira] [Updated] (HBASE-24973) Remove read point parameter in method StoreFlush#performFlush and StoreFlush#createScanner
[ 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#createScanner can also remove this parameter 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} Parameter smallestReadPoint is not used in this method. When `smallestReadPoint` is removed, inner method `createScanner` can remove this necessary parameter too was: Currently, read point parameter in method StoreFlush#performFlush is useless and can be safely removed. and then method StoreFlush#createScanner can also remove this parameter 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} Parameter smallestReadPoint is not used in this method. > 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#createScanner can also remove this parameter > 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} > Parameter smallestReadPoint is not used in this method. When > `smallestReadPoint` is removed, inner method `createScanner` can remove this > necessary parameter too -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (HBASE-24973) Remove read point parameter in method StoreFlush#performFlush and StoreFlush#createScanner
[ 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#createScanner can also remove this parameter 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} Parameter smallestReadPoint is not used in this method. was: 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} > 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#createScanner can also remove this parameter > 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} > Parameter smallestReadPoint is not used in this method. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Updated] (HBASE-24973) Remove read point parameter in method StoreFlush#performFlush and StoreFlush#createScanner
[ https://issues.apache.org/jira/browse/HBASE-24973?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] yuqi updated HBASE-24973: - Summary: Remove read point parameter in method StoreFlush#performFlush and StoreFlush#createScanner (was: Remove read point parameter in StoreFlush#performFlush) > 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. > 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)
[jira] [Updated] (HBASE-24973) Remove read point parameter in method StoreFlush#performFlush and StoreFlush#createScanner
[ 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)