use lambda instead of std::bind (#1243)

Summary:
std::bind(yield_condition, table) will generate a functor which size is larger than std::function's local buf, thus std::function needs to new/delete memory to store the functor.

This PR use the lambda which just capture one pointer(table), which size can fit into std::function's local buf thus new/delete is not needed.

Pull Request resolved: #1243

Reviewed By: lth

Differential Revision: D40858532

Pulled By: hermanlee

fbshipit-source-id: f7431f7

sql/handler.cc

@@ -2912,7 +2912,7 @@ void handler::ha_statistic_increment(
     (table->in_use->status_var.*offset)++;
     table->in_use->check_limit_rows_examined();
     table->in_use->update_sql_stats_periodic();
-    table->in_use->check_yield(std::bind(yield_condition, table));
+    table->in_use->check_yield([t = table] { return yield_condition(t); });
   }
 }
 
@@ -6502,7 +6502,7 @@ ha_rows handler::multi_range_read_info_const(uint keyno, RANGE_SEQ_IF *seq,
   while (!seq->next(seq_it, &range)) {
     if (unlikely(thd->killed != 0)) return HA_POS_ERROR;
 
-    thd->check_yield(std::bind(yield_condition, table));
+    thd->check_yield([t = table] { return yield_condition(t); });
 
     n_ranges++;
     key_range *min_endp, *max_endp;

sql/sql_class.cc

@@ -2733,7 +2733,7 @@ void THD::restore_sub_statement_state(Sub_statement_state *backup) {
 }
 
 void THD::check_yield(std::function<bool()> cond) {
-  yield_cond = cond;
+  yield_cond = std::move(cond);
   thd_wait_begin(this, THD_WAIT_YIELD);
   thd_wait_end(this);
   yield_cond = nullptr;