Henry,

Am 19.05.2015 um 13:06 schrieb Henrik Johansen <henrik.s.johansen@veloxit.no>:

Hi!
I made some (imho) improvements to the OID implementation, if someone with write access would find time to review/commit the attached package, I'd be grateful.

The commit message:
Make OID implementation more compatible with the description in http://docs.mongodb.org/manual/reference/object-id/.

- Use the UNIX epoch instead of Smalltalk epoch for timestamp part, store BE so it's pseudo-sortable by time (also, you can use getTimestamp() in mongo shell and get a sensible value)

Agreed with unix epoch. The old value was in BE also which seems to be the right choice.
 
- Use a per-session ID instead of localhost name for machine identification. localhost name is far from unique on some platforms (always 'localhost' on Macs), better with a unique ID per image session.

I do not agree. "NetNameResolver localHostName" does not return localhost for me. And I'd rather think the NetNameResolver does not work properly on Macs (not returning gethostname) instead of taking it as a reason to tweak client code. The id per Session wouldn't have any value because there is also a process id which gives you more or less the same thing.

-  Counter initialized with a random value.

Good idea.

As an added bonus, [OID new] bench should see a ~10%  increase in performance.

Even better. :) 

What do you think?

I can make you a commiter on the MongoTalk repo. What's your username?

Norbert

Cheers,
Henry
<Mongo-BSON-HenrikSperreJohansen.43.mcz>

On 08 Jul 2013, at 2:45 , Fran��ois Stephany <tulipe.moutarde@gmail.com> wrote:

Hi,

I've just created a new team for MongoTalk on SmalltalkHub:

http://www.smalltalkhub.com/#!/~MongoTalkTeam

The old repo (http://www.smalltalkhub.com/#!/~francois/MongoTalk) will stay just in case some scripts rely on it but you should update to the new repo:

http://www.smalltalkhub.com/#!/~MongoTalkTeam/mongotalk

I've created a version 1.5.1 in the Metacello configuration to reflect the repo migration.
Shout at me if I missed something!

Cheers,
Fa

(Many thanks to Norbert for pushing the migration ;)