[t:4875] Removed cruft from ft (dictionary_opened). Moved options in...

[t:4875] Removed cruft from ft (dictionary_opened).  Moved options in ft_handle to an options struct

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

ft/ft-internal.h

@@ -350,7 +350,6 @@ struct ft {
                                 // counter is effectively a boolean which alternates with each checkpoint.
     LSN checkpoint_lsn;         // LSN of creation of "checkpoint-begin" record in log.  
     int dirty;
-    BOOL dictionary_opened;     // True once this header has been associated with a dictionary (a brt fully opened)
     DICTIONARY_ID dict_id;      // unique id for dictionary
     int panic;                  // If nonzero there was a write error.  Don't write any more, because it probably only gets worse.  This is the error code.
     char *panic_string;         // A malloced string that can indicate what went wrong.
@@ -409,21 +408,25 @@ static inline void setup_fake_db (DB *fake_db, DESCRIPTOR orig_desc) {
 }
 #define FAKE_DB(db, desc) struct __toku_db db; setup_fake_db(&db, (desc))
 
-struct ft_handle {
-    // The header is shared.  It is also ephemeral.
-    FT h;
-
+struct ft_options {
     unsigned int nodesize;
     unsigned int basementnodesize;
     enum toku_compression_method compression_method;
     unsigned int flags;
-    BOOL did_set_flags;
     ft_compare_func compare_fun;
     ft_update_func update_fun;
-    on_redirect_callback redirect_callback;
-    void* redirect_callback_extra;
+};
 
+struct ft_handle {
+    // The header is shared.  It is also ephemeral.
+    FT h;
+
+    on_redirect_callback redirect_callback;
+    void *redirect_callback_extra;
     struct toku_list live_ft_handle_link;
+    BOOL did_set_flags;
+
+    struct ft_options options;
 };
 
 // FIXME needs toku prefix

ft/ft-test-helpers.c

@@ -203,7 +203,7 @@ int toku_testsetup_insert_to_nonleaf (FT_HANDLE brt, BLOCKNUM blocknum, enum ft_
     DBT k;
     int childnum = toku_ftnode_which_child(node,
                                             toku_fill_dbt(&k, key, keylen),
-                                            &brt->h->cmp_descriptor, brt->compare_fun);
+                                            &brt->h->cmp_descriptor, brt->h->compare_fun);
 
     XIDS xids_0 = xids_get_root_xids();
     MSN msn = next_dummymsn();

ft/ft-verify.c

@@ -19,7 +19,7 @@
 static int 
 compare_pairs (FT_HANDLE brt, const DBT *a, const DBT *b) {
     FAKE_DB(db, &brt->h->cmp_descriptor);
-    int cmp = brt->compare_fun(&db, a, b);
+    int cmp = brt->h->compare_fun(&db, a, b);
     return cmp;
 }
 
@@ -27,7 +27,7 @@ static int
 compare_leafentries (FT_HANDLE brt, LEAFENTRY a, LEAFENTRY b) {
     DBT x,y;
     FAKE_DB(db, &brt->h->cmp_descriptor);
-    int cmp = brt->compare_fun(&db,
+    int cmp = brt->h->compare_fun(&db,
                                toku_fill_dbt(&x, le_key(a), le_keylen(a)),
                                toku_fill_dbt(&y, le_key(b), le_keylen(b)));
     return cmp;
@@ -37,7 +37,7 @@ static int
 compare_pair_to_leafentry (FT_HANDLE brt, const DBT *a, LEAFENTRY b) {
     DBT y;
     FAKE_DB(db, &brt->h->cmp_descriptor);
-    int cmp = brt->compare_fun(&db, a, toku_fill_dbt(&y, le_key(b), le_keylen(b)));
+    int cmp = brt->h->compare_fun(&db, a, toku_fill_dbt(&y, le_key(b), le_keylen(b)));
     return cmp;
 }
 
@@ -45,7 +45,7 @@ static int
 compare_pair_to_key (FT_HANDLE brt, const DBT *a, bytevec key, ITEMLEN keylen) {
     DBT y;
     FAKE_DB(db, &brt->h->cmp_descriptor);
-    int cmp = brt->compare_fun(&db, a, toku_fill_dbt(&y, key, keylen));
+    int cmp = brt->h->compare_fun(&db, a, toku_fill_dbt(&y, key, keylen));
     return cmp;
 }
 
@@ -171,7 +171,7 @@ verify_sorted_by_key_msn(FT_HANDLE brt, FIFO fifo, OMT mt) {
         assert_zero(r);
         size_t offset = (size_t) v;
         if (i > 0) {
-            struct toku_fifo_entry_key_msn_cmp_extra extra = { .desc = &brt->h->cmp_descriptor, .cmp = brt->compare_fun, .fifo = fifo };
+            struct toku_fifo_entry_key_msn_cmp_extra extra = { .desc = &brt->h->cmp_descriptor, .cmp = brt->h->compare_fun, .fifo = fifo };
             if (toku_fifo_entry_key_msn_cmp(&extra, &last_offset, &offset) >= 0) {
                 result = TOKUDB_NEEDS_REPAIR;
                 break;
@@ -185,7 +185,7 @@ verify_sorted_by_key_msn(FT_HANDLE brt, FIFO fifo, OMT mt) {
 static int
 count_eq_key_msn(FT_HANDLE brt, FIFO fifo, OMT mt, const DBT *key, MSN msn) {
     struct toku_fifo_entry_key_msn_heaviside_extra extra = { 
-        .desc = &brt->h->cmp_descriptor, .cmp = brt->compare_fun, .fifo = fifo, .key = key, .msn = msn 
+        .desc = &brt->h->cmp_descriptor, .cmp = brt->h->compare_fun, .fifo = fifo, .key = key, .msn = msn 
     };
     OMTVALUE v; u_int32_t idx;
     int r = toku_omt_find_zero(mt, toku_fifo_entry_key_msn_heaviside, &extra, &v, &idx);
@@ -311,7 +311,7 @@ toku_verify_ftnode (FT_HANDLE brt,
                              }
                              struct count_msgs_extra extra = { .count = 0, .key = &keydbt,
                                                                .msn = msn, .fifo = bnc->buffer,
-                                                               .desc = &brt->h->cmp_descriptor, .cmp = brt->compare_fun };
+                                                               .desc = &brt->h->cmp_descriptor, .cmp = brt->h->compare_fun };
                              extra.count = 0;
                              toku_omt_iterate(bnc->broadcast_list, count_msgs, &extra);
                              if (ft_msg_type_applies_all(type) || ft_msg_type_does_nothing(type)) {

ft/ft.c

@@ -254,7 +254,7 @@ ft_close (CACHEFILE cachefile, int fd, void *header_v, char **malloced_error_str
     int r = 0;
     if (h->panic) {
         r = h->panic;
-    } else if (h->dictionary_opened) { //Otherwise header has never fully been created.
+    } else {
         assert(h->cf == cachefile);
         TOKULOGGER logger = toku_cachefile_logger(cachefile);
         LSN lsn = ZERO_LSN;
@@ -357,33 +357,40 @@ static int setup_initial_ft_root_node (FT h, BLOCKNUM blocknum) {
     return r;
 }
 
-// TODO: (Zardosht) move this functionality to ft_init
-// No need in having ft_init call this function
 static int
-ft_init_partial (FT_HANDLE t, CACHEFILE cf, TOKUTXN txn) {
-    int r;
-    t->h->flags = t->flags;
-    if (t->h->cf!=NULL) assert(t->h->cf == cf);
-    t->h->cf = cf;
-    t->h->nodesize = t->nodesize;
-    t->h->basementnodesize = t->basementnodesize;
-    t->h->compression_method = t->compression_method;
-    t->h->root_xid_that_created = txn ? txn->ancestor_txnid64 : TXNID_NONE;
-    t->h->compare_fun = t->compare_fun;
-    t->h->update_fun = t->update_fun;
-    t->h->in_memory_stats          = ZEROSTATS;
-    t->h->on_disk_stats            = ZEROSTATS;
-    t->h->checkpoint_staging_stats = ZEROSTATS;
-    t->h->highest_unused_msn_for_upgrade.msn = MIN_MSN.msn - 1;
-
-    BLOCKNUM root = t->h->root_blocknum;
-    r = setup_initial_ft_root_node(t->h, root);
+ft_init (FT ft, FT_OPTIONS options, CACHEFILE cf, TOKUTXN txn) {
+    ft->type = FT_CURRENT;
+    ft->checkpoint_header = NULL;
+    toku_ft_init_treelock(ft);
+    toku_blocktable_create_new(&ft->blocktable);
+    //Assign blocknum for root block, also dirty the header
+    toku_allocate_blocknum(ft->blocktable, &ft->root_blocknum, ft);
+
+    toku_list_init(&ft->live_ft_handles);
+    int r = toku_omt_create(&ft->txns);
+    assert_zero(r);
+    ft->flags = options->flags;
+    ft->nodesize = options->nodesize;
+    ft->basementnodesize = options->basementnodesize;
+    ft->compression_method = options->compression_method;
+    ft->compare_fun = options->compare_fun;
+    ft->update_fun = options->update_fun;
+
+    if (ft->cf!=NULL) assert(ft->cf == cf);
+    ft->cf = cf;
+    ft->root_xid_that_created = txn ? txn->ancestor_txnid64 : TXNID_NONE;
+    ft->in_memory_stats          = ZEROSTATS;
+    ft->on_disk_stats            = ZEROSTATS;
+    ft->checkpoint_staging_stats = ZEROSTATS;
+    ft->highest_unused_msn_for_upgrade.msn = MIN_MSN.msn - 1;
+
+    r = setup_initial_ft_root_node(ft, ft->root_blocknum);
     if (r != 0) { 
         goto exit; 
     }
-    //printf("%s:%d putting %p (%d)\n", __FILE__, __LINE__, t->h, 0);
-    toku_cachefile_set_userdata(t->h->cf,
-                                t->h,
+    //printf("%s:%d putting %p (%d)\n", __FILE__, __LINE__, ft, 0);
+    toku_cachefile_set_userdata(ft->cf,
+                                ft,
                                 ft_log_fassociate_during_checkpoint,
                                 ft_log_suppress_rollback_during_checkpoint,
                                 ft_close,
@@ -392,67 +399,51 @@ ft_init_partial (FT_HANDLE t, CACHEFILE cf, TOKUTXN txn) {
                                 ft_end_checkpoint,
                                 ft_note_pin_by_checkpoint,
                                 ft_note_unpin_by_checkpoint);
-exit:
-    return r;
-}
 
-static int
-ft_init (FT_HANDLE t, CACHEFILE cf, TOKUTXN txn) {
-    t->h->type = FT_CURRENT;
-    t->h->checkpoint_header = NULL;
-    toku_ft_init_treelock(t->h);
-    toku_blocktable_create_new(&t->h->blocktable);
-    BLOCKNUM root;
-    //Assign blocknum for root block, also dirty the header
-    toku_allocate_blocknum(t->h->blocktable, &root, t->h);
-    t->h->root_blocknum = root;
-
-    toku_list_init(&t->h->live_ft_handles);
-    int r = toku_omt_create(&t->h->txns);
-    assert_zero(r);
-    r = ft_init_partial(t, cf, txn);
-    if (r==0) toku_block_verify_no_free_blocknums(t->h->blocktable);
+    toku_block_verify_no_free_blocknums(ft->blocktable);
+    r = 0;
+exit:
     return r;
 }
 
 
 // allocate and initialize a brt header.
 // t->h->cf is not set to anything.
-// TODO: (Zardosht) make this function return a header and set
-// it to t->h in the caller
 int 
-toku_create_new_ft(FT_HANDLE t, CACHEFILE cf, TOKUTXN txn) {
+toku_create_new_ft(FT *ftp, FT_OPTIONS options, CACHEFILE cf, TOKUTXN txn) {
     int r;
+    invariant(ftp);
+
+    FT XCALLOC(ft);
     
-    assert (!t->h);
-    XCALLOC(t->h);
 
-    t->h->layout_version = FT_LAYOUT_VERSION;
-    t->h->layout_version_original = FT_LAYOUT_VERSION;
-    t->h->layout_version_read_from_disk = FT_LAYOUT_VERSION;             // fake, prevent unnecessary upgrade logic
+    ft->layout_version = FT_LAYOUT_VERSION;
+    ft->layout_version_original = FT_LAYOUT_VERSION;
+    ft->layout_version_read_from_disk = FT_LAYOUT_VERSION;             // fake, prevent unnecessary upgrade logic
 
-    t->h->build_id = BUILD_ID;
-    t->h->build_id_original = BUILD_ID;
+    ft->build_id = BUILD_ID;
+    ft->build_id_original = BUILD_ID;
 
     uint64_t now = (uint64_t) time(NULL);
-    t->h->time_of_creation = now;
-    t->h->time_of_last_modification = now;
-    t->h->time_of_last_verification = 0;
+    ft->time_of_creation = now;
+    ft->time_of_last_modification = now;
+    ft->time_of_last_verification = 0;
 
-    memset(&t->h->descriptor, 0, sizeof(t->h->descriptor));
-    memset(&t->h->cmp_descriptor, 0, sizeof(t->h->cmp_descriptor));
+    memset(&ft->descriptor, 0, sizeof(ft->descriptor));
+    memset(&ft->cmp_descriptor, 0, sizeof(ft->cmp_descriptor));
 
-    r = ft_init(t, cf, txn);
+    r = ft_init(ft, options, cf, txn);
     if (r != 0) {
         goto exit;
     }
 
+    *ftp = ft;
     r = 0;
 exit:
     if (r != 0) {
-        if (t->h) {
-            toku_free(t->h);
-            t->h = NULL;
+        if (ft) {
+            toku_free(ft);
+            ft = NULL;
         }
         return r;
     }
@@ -470,8 +461,8 @@ int toku_read_ft_and_store_in_cachefile (FT_HANDLE brt, CACHEFILE cf, LSN max_ac
         if ((h=toku_cachefile_get_userdata(cf))!=0) {
             *header = h;
             *was_open = TRUE;
-            assert(brt->update_fun == h->update_fun);
-            assert(brt->compare_fun == h->compare_fun);
+            assert(brt->options.update_fun == h->update_fun);
+            assert(brt->options.compare_fun == h->compare_fun);
             return 0;
         }
     }
@@ -495,8 +486,8 @@ int toku_read_ft_and_store_in_cachefile (FT_HANDLE brt, CACHEFILE cf, LSN max_ac
     }
     if (r!=0) return r;
     h->cf = cf;
-    h->compare_fun = brt->compare_fun;
-    h->update_fun = brt->update_fun;
+    h->compare_fun = brt->options.compare_fun;
+    h->update_fun = brt->options.update_fun;
     toku_cachefile_set_userdata(cf,
                                 (void*)h,
                                 ft_log_fassociate_during_checkpoint,
@@ -512,12 +503,11 @@ int toku_read_ft_and_store_in_cachefile (FT_HANDLE brt, CACHEFILE cf, LSN max_ac
 }
 
 void
-toku_ft_note_ft_handle_open(FT_HANDLE live) {
-    FT h = live->h;
-    toku_ft_lock(h);
-    toku_list_push(&h->live_ft_handles, &live->live_ft_handle_link);
-    h->dictionary_opened = TRUE;
-    toku_ft_unlock(h);
+toku_ft_note_ft_handle_open(FT ft, FT_HANDLE live) {
+    toku_ft_lock(ft);
+    live->h = ft;
+    toku_list_push(&ft->live_ft_handles, &live->live_ft_handle_link);
+    toku_ft_unlock(ft);
 }
 
 int
@@ -674,18 +664,16 @@ dictionary_redirect_internal(const char *dst_fname_in_env, FT src_h, TOKUTXN txn
     // we want to change it to dummy_dst
     while (!toku_list_empty(&src_h->live_ft_handles)) {
         list = src_h->live_ft_handles.next;
-        FT_HANDLE src_ft = NULL;
-        src_ft = toku_list_struct(list, struct ft_handle, live_ft_handle_link);
+        FT_HANDLE src_handle = NULL;
+        src_handle = toku_list_struct(list, struct ft_handle, live_ft_handle_link);
 
         toku_ft_lock(src_h);
-        toku_list_remove(&src_ft->live_ft_handle_link);
+        toku_list_remove(&src_handle->live_ft_handle_link);
         toku_ft_unlock(src_h);
         
-        src_ft->h = dst_h;
-        
-        toku_ft_note_ft_handle_open(src_ft);
-        if (src_ft->redirect_callback) {
-            src_ft->redirect_callback(src_ft, src_ft->redirect_callback_extra);
+        toku_ft_note_ft_handle_open(dst_h, src_handle);
+        if (src_handle->redirect_callback) {
+            src_handle->redirect_callback(src_handle, src_handle->redirect_callback_extra);
         }
     }
     assert(dst_h);

ft/ft.h

@@ -22,11 +22,11 @@ void toku_ft_destroy_treelock(FT h);
 void toku_ft_grab_treelock(FT h);
 void toku_ft_release_treelock(FT h);
 
-int toku_create_new_ft(FT_HANDLE t, CACHEFILE cf, TOKUTXN txn);
+int toku_create_new_ft(FT *ftp, FT_OPTIONS options, CACHEFILE cf, TOKUTXN txn);
 void toku_ft_free (FT h);
 
 int toku_read_ft_and_store_in_cachefile (FT_HANDLE brt, CACHEFILE cf, LSN max_acceptable_lsn, FT *header, BOOL* was_open);
-void toku_ft_note_ft_handle_open(FT_HANDLE live);
+void toku_ft_note_ft_handle_open(FT ft, FT_HANDLE live);
 
 int toku_ft_needed(FT h);
 int toku_remove_ft (FT h, char **error_string, BOOL oplsn_valid, LSN oplsn)  __attribute__ ((warn_unused_result));

ft/fttypes.h

@@ -38,6 +38,7 @@ typedef struct ftnode_leaf_basement_node *BASEMENTNODE;
 typedef struct ftnode_nonleaf_childinfo *NONLEAF_CHILDINFO;
 typedef struct sub_block *SUB_BLOCK;
 typedef struct ft *FT;
+typedef struct ft_options *FT_OPTIONS;
 struct wbuf;
 struct dbuf;
 

ft/tests/make-tree.c

@@ -66,7 +66,7 @@ insert_into_child_buffer(FT_HANDLE brt, FTNODE node, int childnum, int minkey, i
         unsigned int key = htonl(val);
         DBT thekey; toku_fill_dbt(&thekey, &key, sizeof key);
         DBT theval; toku_fill_dbt(&theval, &val, sizeof val);
-        toku_ft_append_to_child_buffer(brt->compare_fun, NULL, node, childnum, FT_INSERT, msn, xids_get_root_xids(), true, &thekey, &theval);
+        toku_ft_append_to_child_buffer(brt->h->compare_fun, NULL, node, childnum, FT_INSERT, msn, xids_get_root_xids(), true, &thekey, &theval);
 	node->max_msn_applied_to_node_on_disk = msn;
     }
 }

ft/tests/msnfilter.c

@@ -48,7 +48,7 @@ append_leaf(FT_HANDLE brt, FTNODE leafnode, void *key, size_t keylen, void *val,
     FT_MSG_S cmd = { FT_INSERT, msn, xids_get_root_xids(), .u.id = { &thekey, &theval } };
 
     u_int64_t workdone=0;
-    toku_ft_leaf_apply_cmd(brt->compare_fun, brt->update_fun, &brt->h->cmp_descriptor, leafnode, &cmd, &workdone, NULL);
+    toku_ft_leaf_apply_cmd(brt->h->compare_fun, brt->h->update_fun, &brt->h->cmp_descriptor, leafnode, &cmd, &workdone, NULL);
     {
 	int r = toku_ft_lookup(brt, &thekey, lookup_checkf, &pair);
 	assert(r==0);
@@ -56,7 +56,7 @@ append_leaf(FT_HANDLE brt, FTNODE leafnode, void *key, size_t keylen, void *val,
     }
 
     FT_MSG_S badcmd = { FT_INSERT, msn, xids_get_root_xids(), .u.id = { &thekey, &badval } };
-    toku_ft_leaf_apply_cmd(brt->compare_fun, brt->update_fun, &brt->h->cmp_descriptor, leafnode, &badcmd, &workdone, NULL);
+    toku_ft_leaf_apply_cmd(brt->h->compare_fun, brt->h->update_fun, &brt->h->cmp_descriptor, leafnode, &badcmd, &workdone, NULL);
 
     
     // message should be rejected for duplicate msn, row should still have original val
@@ -69,7 +69,7 @@ append_leaf(FT_HANDLE brt, FTNODE leafnode, void *key, size_t keylen, void *val,
     // now verify that message with proper msn gets through
     msn = next_dummymsn();
     FT_MSG_S cmd2 = { FT_INSERT, msn, xids_get_root_xids(), .u.id = { &thekey, &val2 } };
-    toku_ft_leaf_apply_cmd(brt->compare_fun, brt->update_fun, &brt->h->cmp_descriptor, leafnode, &cmd2, &workdone, NULL);
+    toku_ft_leaf_apply_cmd(brt->h->compare_fun, brt->h->update_fun, &brt->h->cmp_descriptor, leafnode, &cmd2, &workdone, NULL);
     
     // message should be accepted, val should have new value
     {
@@ -81,7 +81,7 @@ append_leaf(FT_HANDLE brt, FTNODE leafnode, void *key, size_t keylen, void *val,
     // now verify that message with lesser (older) msn is rejected
     msn.msn = msn.msn - 10;
     FT_MSG_S cmd3 = { FT_INSERT, msn, xids_get_root_xids(), .u.id = { &thekey, &badval } };
-    toku_ft_leaf_apply_cmd(brt->compare_fun, brt->update_fun, &brt->h->cmp_descriptor, leafnode, &cmd3, &workdone, NULL);
+    toku_ft_leaf_apply_cmd(brt->h->compare_fun, brt->h->update_fun, &brt->h->cmp_descriptor, leafnode, &cmd3, &workdone, NULL);
     
     // message should be rejected, val should still have value in pair2
     {

ft/tests/orthopush-flush.c

@@ -126,7 +126,7 @@ insert_random_message_to_bn(FT_HANDLE t, BASEMENTNODE blb, LEAFENTRY *save, XIDS
     int64_t numbytes;
     int r = apply_msg_to_leafentry(&msg, NULL, &memsize, save, NULL, NULL, NULL, &numbytes);
     assert_zero(r);
-    toku_ft_bn_apply_cmd(t->compare_fun, t->update_fun, NULL, blb, &msg, NULL, NULL);
+    toku_ft_bn_apply_cmd(t->h->compare_fun, t->h->update_fun, NULL, blb, &msg, NULL, NULL);
     if (msn.msn > blb->max_msn_applied.msn) {
         blb->max_msn_applied = msn;
     }
@@ -167,11 +167,11 @@ insert_same_message_to_bns(FT_HANDLE t, BASEMENTNODE blb1, BASEMENTNODE blb2, LE
     int64_t numbytes;
     int r = apply_msg_to_leafentry(&msg, NULL, &memsize, save, NULL, NULL, NULL, &numbytes);
     assert_zero(r);
-    toku_ft_bn_apply_cmd(t->compare_fun, t->update_fun, NULL, blb1, &msg, NULL, NULL);
+    toku_ft_bn_apply_cmd(t->h->compare_fun, t->h->update_fun, NULL, blb1, &msg, NULL, NULL);
     if (msn.msn > blb1->max_msn_applied.msn) {
         blb1->max_msn_applied = msn;
     }
-    toku_ft_bn_apply_cmd(t->compare_fun, t->update_fun, NULL, blb2, &msg, NULL, NULL);
+    toku_ft_bn_apply_cmd(t->h->compare_fun, t->h->update_fun, NULL, blb2, &msg, NULL, NULL);
     if (msn.msn > blb2->max_msn_applied.msn) {
         blb2->max_msn_applied = msn;
     }
@@ -583,7 +583,7 @@ flush_to_leaf(FT_HANDLE t, bool make_leaf_up_to_date, bool use_flush) {
     if (make_leaf_up_to_date) {
         for (i = 0; i < num_parent_messages; ++i) {
             if (!parent_messages_is_fresh[i]) {
-                toku_ft_leaf_apply_cmd(t->compare_fun, t->update_fun, &t->h->descriptor, child, parent_messages[i], NULL, NULL);
+                toku_ft_leaf_apply_cmd(t->h->compare_fun, t->h->update_fun, &t->h->descriptor, child, parent_messages[i], NULL, NULL);
             }
         }
         for (i = 0; i < 8; ++i) {
@@ -807,7 +807,7 @@ flush_to_leaf_with_keyrange(FT_HANDLE t, bool make_leaf_up_to_date) {
         for (i = 0; i < num_parent_messages; ++i) {
             if (dummy_cmp(NULL, parent_messages[i]->u.id.key, &childkeys[7]) <= 0 &&
                 !parent_messages_is_fresh[i]) {
-                toku_ft_leaf_apply_cmd(t->compare_fun, t->update_fun, &t->h->descriptor, child, parent_messages[i], NULL, NULL);
+                toku_ft_leaf_apply_cmd(t->h->compare_fun, t->h->update_fun, &t->h->descriptor, child, parent_messages[i], NULL, NULL);
             }
         }
         for (i = 0; i < 8; ++i) {
@@ -1000,8 +1000,8 @@ compare_apply_and_flush(FT_HANDLE t, bool make_leaf_up_to_date) {
     if (make_leaf_up_to_date) {
         for (i = 0; i < num_parent_messages; ++i) {
             if (!parent_messages_is_fresh[i]) {
-                toku_ft_leaf_apply_cmd(t->compare_fun, t->update_fun, &t->h->descriptor, child1, parent_messages[i], NULL, NULL);
-                toku_ft_leaf_apply_cmd(t->compare_fun, t->update_fun, &t->h->descriptor, child2, parent_messages[i], NULL, NULL);
+                toku_ft_leaf_apply_cmd(t->h->compare_fun, t->h->update_fun, &t->h->descriptor, child1, parent_messages[i], NULL, NULL);
+                toku_ft_leaf_apply_cmd(t->h->compare_fun, t->h->update_fun, &t->h->descriptor, child2, parent_messages[i], NULL, NULL);
             }
         }
         for (i = 0; i < 8; ++i) {

ft/tests/test-dirty-flushes-on-cleaner.c

@@ -59,7 +59,7 @@ doit (void) {
     assert(r==0);
     toku_free(fname);
 
-    brt->update_fun = update_func;
+    brt->h->update_fun = update_func;
     brt->h->update_fun = update_func;
     
     toku_testsetup_initialize();  // must precede any other toku_testsetup calls

ft/tests/test-flushes-on-cleaner.c

@@ -59,7 +59,7 @@ doit (BOOL keep_other_bn_in_memory) {
     assert(r==0);
     toku_free(fname);
 
-    brt->update_fun = update_func;
+    brt->options.update_fun = update_func;
     brt->h->update_fun = update_func;
     
     toku_testsetup_initialize();  // must precede any other toku_testsetup calls

ft/tests/test-merges-on-cleaner.c

@@ -58,7 +58,7 @@ doit (void) {
     assert(r==0);
     toku_free(fname);
 
-    brt->update_fun = update_func;
+    brt->options.update_fun = update_func;
     brt->h->update_fun = update_func;
     
     toku_testsetup_initialize();  // must precede any other toku_testsetup calls

ft/tests/verify-bad-msn.c

@@ -70,7 +70,7 @@ insert_into_child_buffer(FT_HANDLE brt, FTNODE node, int childnum, int minkey, i
         unsigned int key = htonl(val);
         DBT thekey; toku_fill_dbt(&thekey, &key, sizeof key);
         DBT theval; toku_fill_dbt(&theval, &val, sizeof val);
-        toku_ft_append_to_child_buffer(brt->compare_fun, NULL, node, childnum, FT_INSERT, msn, xids_get_root_xids(), true, &thekey, &theval);
+        toku_ft_append_to_child_buffer(brt->h->compare_fun, NULL, node, childnum, FT_INSERT, msn, xids_get_root_xids(), true, &thekey, &theval);
 
 	// Create bad tree (don't do following):
 	// node->max_msn_applied_to_node = msn;

ft/tests/verify-misrouted-msgs.c

@@ -58,7 +58,7 @@ insert_into_child_buffer(FT_HANDLE brt, FTNODE node, int childnum, int minkey, i
         DBT thekey; toku_fill_dbt(&thekey, &key, sizeof key);
         DBT theval; toku_fill_dbt(&theval, &val, sizeof val);
 	MSN msn = next_dummymsn();
-        toku_ft_append_to_child_buffer(brt->compare_fun, NULL, node, childnum, FT_INSERT, msn, xids_get_root_xids(), true, &thekey, &theval);
+        toku_ft_append_to_child_buffer(brt->h->compare_fun, NULL, node, childnum, FT_INSERT, msn, xids_get_root_xids(), true, &thekey, &theval);
     }
 }
 

src/ydb_db.h

@@ -34,7 +34,7 @@ static inline int db_opened(DB *db) {
 
 static inline toku_dbt_cmp 
 toku_db_get_compare_fun(DB* db) {
-    return db->i->ft_handle->compare_fun;
+    return db->i->ft_handle->h->compare_fun;
 }
 
 int toku_db_pre_acquire_fileops_lock(DB *db, DB_TXN *txn);