Ok. That's what Jay is talking about and the reason he advocates
drizzled::algorithm.

I am saying I don't have a problem with drizzled::algorithm for things
like sorting and searching, since that's the sort of things you'll find
in <algorithm> in the STL. But something that just computes a hash
doesn't belong lumped in with code that searches a container to me.


Jay Pipes wrote:
> Understood, but we are talking about algorithms that are not necessarily
> restricted to just hashing.  For instance, sorting and searching
> algorithms.  We are asking whether drizzled::algorithm:: is an
> appropriate namespace for this work to go into?  Or should it be any of
> the following?
> 
> pandora::algorithm::
> pandora::util::
> drizzled::util::
> 
> Something else?
> 
> -jay
> 
> Brian Aker wrote:
>> Hi!
>>
>> On Jan 5, 2010, at 11:52 AM, Joe Daly wrote:
>>
>>> Currently what resides in this namespace is the crc32 implementation
>>> used for md5 checksums. 
>>
>> The plan is to copy in libhashkit at some point (or use it if it is
>> locally installed). We are aways off from this though, but that is the
>> eventual plan. I want to be able to re-use all of the
>> hash/distribution work in that library.
>>
>> Cheers,
>>     -Brian
>> _______________________________________________
>> 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
> 
> _______________________________________________
> 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
> 


_______________________________________________
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

Reply via email to