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 <henrik.i...@avoinelama.fi>wrote: > On Fri, Jul 13, 2012 at 4:09 AM, Stewart Smith <stew...@flamingspork.com> > 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 is going to be used. I would set this at 64. > >> (It should be even higher, but then we'd need to implement something > >> that creates more threads as needed, not just 1024 threads at > >> startup.) > > > > If we can easily get number of CPU cores, let's just launch that many > > threads. If each thread is for the large part non-blocking, then we're > > going to be pretty sweet with this as default. > > I think it is better if the default is the same on all servers. > > In the future maybe someone implements code where we don't need to > launch all threads at startup. In that case the number of threads > *actually created* could be deduced by such logic. But even then, the > default value of max_threads should be always the same. (Also, it > should then be much higher than number of cpu's, since it is then a > real maximum, not the actual number of threads.) > > >> I've been thinking you could just remove all the /0.1/* and /0.2/ and > >> /latest/ URIs. We are not strictly leaving old versions of the > >> functions in place (also because they had crashing bugs) so let's just > >> simplify and just provide /version /sql and /json and nothing more. > > > > I'd like if we did keep the version numbers... it allows us to have some > > form of backwards compat. > > > > e.g. if we remove 0.1/ because it's buggy and useless, and somebody has > > an app, a 404 error is much better than failing weirdly. > > > > I did it this way to begin with very much on purpose so that others > > could come along with much better API ideas and that any users could > > either continue to be happy with old APIs or get a very simple error > > message if we removed them. > > 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 crashing bug, unless we fix it soon, > also that will be removed in the future...) > > henrik > -- > henrik.i...@avoinelama.fi > +358-40-8211286 skype: henrik.ingo irc: hingo > www.openlife.cc > > My LinkedIn profile: http://www.linkedin.com/profile/view?id=9522559 >
_______________________________________________ Mailing list: https://launchpad.net/~drizzle-discuss Post to : drizzle-discuss@lists.launchpad.net Unsubscribe : https://launchpad.net/~drizzle-discuss More help : https://help.launchpad.net/ListHelp