• Y
    Enhance partial merge to support multiple arguments · cda4006e
    Yueh-Hsuan Chiang 提交于
    Summary:
    * PartialMerge api now takes a list of operands instead of two operands.
    * Add min_pertial_merge_operands to Options, indicating the minimum
      number of operands to trigger partial merge.
    * This diff is based on Schalk's previous diff (D14601), but it also
      includes necessary changes such as updating the pure C api for
      partial merge.
    
    Test Plan:
    * make check all
    * develop tests for cases where partial merge takes more than two
      operands.
    
    TODOs (from Schalk):
    * Add test with min_partial_merge_operands > 2.
    * Perform benchmarks to measure the performance improvements (can probably
      use results of task #2837810.)
    * Add description of problem to doc/index.html.
    * Change wiki pages to reflect the interface changes.
    
    Reviewers: haobo, igor, vamsi
    
    Reviewed By: haobo
    
    CC: leveldb, dhruba
    
    Differential Revision: https://reviews.facebook.net/D16815
    cda4006e
c_test.c 13.5 KB