Re: [Drizzle-discuss] Review of json_server multi-threading

2012-07-17 Thread Mohit Srivastava
Hi, Check this: http://bazaar.launchpad.net/~srivastavamohit91/drizzle/drizzle-json_server-multithreading/revision/2572 On Mon, Jul 16, 2012 at 3:39 PM, Henrik Ingo wrote: > On Sat, Jul 14, 2012 at 5:13 PM, Mohit Srivastava > wrote: >> http://bazaar.launchpad.net/~srivastavamohit91/drizzle/drizz

Re: [Drizzle-discuss] Review of json_server multi-threading

2012-07-16 Thread Stewart Smith
Henrik Ingo writes: > In theory yes. In practice both 0.1 and 0.2 versions have bugs that > will crash the server, both in sql/ and in json/. But your proposal > means we will now support the following urls: > > /0.3/* > /latest/* > /sql > /json > /version > / > > (Also /0.3/sql now has a known cr

Re: [Drizzle-discuss] Review of json_server multi-threading

2012-07-16 Thread Henrik Ingo
On Sat, Jul 14, 2012 at 5:13 PM, Mohit Srivastava wrote: > http://bazaar.launchpad.net/~srivastavamohit91/drizzle/drizzle-alsosql-keyvalue/revision/2575 > I am done with multi-threading and dynamic plugin part.New one for review > :) On Thu, Jul 12, 2012 at 11:06 PM, Henrik Ingo wrote: > *** M

Re: [Drizzle-discuss] Review of json_server multi-threading

2012-07-14 Thread Mohit Srivastava
Hi , In previous commit I was forgot to test: http://bazaar.launchpad.net/~srivastavamohit91/drizzle/drizzle-alsosql-keyvalue/revision/2575 -- Mohit On Sat, Jul 14, 2012 at 3:20 PM, Mohit Srivastava < srivastavamohi...@gmail.com> wrote: > Hi Henrik, > I am done with multi-threading and dynamic

Re: [Drizzle-discuss] Review of json_server multi-threading

2012-07-14 Thread Mohit Srivastava
Hi Henrik, I am done with multi-threading and dynamic plugin part.New one for review :) http://bazaar.launchpad.net/~srivastavamohit91/drizzle/drizzle-alsosql-keyvalue/revision/2574 -- Mohit On Fri, Jul 13, 2012 at 2:48 PM, Henrik Ingo wrote: > On Fri, Jul 13, 2012 at 4:09 AM, Stewart Smith >

Re: [Drizzle-discuss] Review of json_server multi-threading

2012-07-13 Thread Henrik Ingo
On Fri, Jul 13, 2012 at 4:09 AM, Stewart Smith wrote: >> Ability to at least increase max_threads dynamically. > > Defaulting to roughly number of CPU cores could be a good default. Ok, 32 then. >> Use a higher default. Json server isn't loaded by default. So if it is >> loaded, we can assume it

Re: [Drizzle-discuss] Review of json_server multi-threading

2012-07-12 Thread Stewart Smith
Henrik Ingo writes: > Review of > https://code.launchpad.net/~srivastavamohit91/drizzle/drizzle-alsosql-keyvalue > Revisions 2570 .. 2573 > > *** Miscellaneous: > > Please use more descriptive name for the branch. > "drizzle-json_server-multihtreading" would be a good choice here. (Or > at least