Skip to content

Commit

Permalink
fix to initialize the default bucket
Browse files Browse the repository at this point in the history
The last commit moved initialization out of load_engine().
So, the default bucket pathway was incorrectly missing initialization
of the proxied bucket.

Change-Id: I01670d9856b261e855f040a1b37fffb260b2734c
Reviewed-on: http://review.northscale.com:8080/345
Reviewed-by: Sean Lynch <seanl@literati.org>
Tested-by: Sean Lynch <seanl@literati.org>
  • Loading branch information
steveyen authored and Sean Lynch committed Jun 9, 2010
1 parent ca0dae9 commit 9d947af
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions bucket_engine.c
Expand Up @@ -463,6 +463,16 @@ static ENGINE_ERROR_CODE bucket_initialize(ENGINE_HANDLE* handle,
se->default_engine.refcount = 1;
se->default_engine.valid = true;
se->default_engine.pe.v0 = load_engine(se->proxied_engine_path, "", NULL);

ENGINE_HANDLE_V1 *dv1 = (ENGINE_HANDLE_V1*)se->default_engine.pe.v0;
if (!dv1) {
return ENGINE_FAILED;
}

if (dv1->initialize(se->default_engine.pe.v0, config_str) != ENGINE_SUCCESS) {
dv1->destroy(se->default_engine.pe.v0);
return ENGINE_FAILED;
}
}

se->server->callback->register_callback(ON_CONNECT, handle_connect, se);
Expand Down

0 comments on commit 9d947af

Please sign in to comment.