You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@hbase.apache.org by GitBox <gi...@apache.org> on 2020/06/25 16:10:55 UTC

[GitHub] [hbase] bsglz removed a comment on pull request #1737: HBASE-24382 Flush partial stores of region filtered by seqId when arc…

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:
users@infra.apache.org