bsglz removed a comment on pull request #1737:
URL: https://github.com/apache/hbase/pull/1737#issuecomment-649647335


   @saintstack 
   When doing the change, I found some places that also set the 
forceFlushAllStores true, such as HRegion.bulkLoadHFiles, HRegion.flush and 
many unit tests.
   If we remove it, we should convert it to a list of all families in all that 
places, seems not a good choise.
   So i think use a enum would be more clear to understand, WDYT?
   Thanks.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to