refs #5197 Fix a couple of bugs (off by true, and missing braces around if...

refs #5197 Fix a couple of bugs (off by true, and missing braces around if statement) with not reserving memory for put-loader
Also only call toku_txn_require_checkpoint_on_commit(tokutxn) once (instead of N times)

git-svn-id: file:///svn/toku/tokudb@45351 c7de825b-a66e-492c-adef-691d508d4ae1

ft/ftloader.c

@@ -374,9 +374,10 @@ void toku_ft_loader_internal_destroy (FTLOADER bl, BOOL is_error) {
     toku_free(bl->fractal_queues);
     toku_free(bl->fractal_threads_live);
 
-    if (bl->did_reserve_memory)
+    if (bl->did_reserve_memory) {
         invariant(bl->cachetable);
         toku_cachetable_release_reserved_memory(bl->cachetable, bl->reserved_memory);
+    }
 
     ft_loader_destroy_error_callback(&bl->error_callback);
     ft_loader_destroy_poll_callback(&bl->poll_callback);

src/indexer.c

@@ -490,9 +490,7 @@ close_indexer(DB_INDEXER *indexer) {
     //   to create them are not in the recovery log.)
     DB_TXN     *txn = indexer->i->txn;
     TOKUTXN tokutxn = db_txn_struct_i(txn)->tokutxn;
-    for (int which_db = 0; which_db < indexer->i->N ; which_db++) {
     toku_txn_require_checkpoint_on_commit(tokutxn);
-    }
 
     // Disassociate the indexer from the hot db and free_indexer
     disassociate_indexer_from_hot_dbs(indexer);

src/loader.c

@@ -274,7 +274,7 @@ int toku_loader_create_loader(DB_ENV *env,
                                  loader->i->temp_file_template,
                                  load_lsn,
                                  ttxn,
-                                 use_puts);
+                                 !use_puts);
         if ( r!=0 ) {
             toku_free(new_inames_in_env);
             toku_free(brts);