continuum-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Trygve Laugstøl <tryg...@apache.org>
Subject Re: [vote] merge id-refactor branch changes
Date Mon, 29 Jan 2007 13:43:14 GMT
Rahul Thakur wrote:
> 
> Trygve Laugstøl wrote:
>> Rahul Thakur wrote:
>>>>> 'int' ids are now converted to 'long' across the project and to 
>>>>> allow really large values. This should cater to scenarios where the 
>>>>> id generation could be started from an arbitrary large value.
>>>>
>>>> Won't this break the API?
>>>
>>> Yep, it would.
>>>
>>>>
>>>> What is the use case where 4 billion IDs isn't sufficient?
>>>
>>> 2 billion you mean :-). But this also more of something that I have 
>>> noticed  'traditionally' that ids are specified as long and stored as 
>>> bigints in database
>>
>> No, 4 billion. an int is +-2billion. Anyway, just because longs are 
>> more traditionally used that is not a good enough reason to switch to 
>> longs and break the API to me.
> 
> Yep, I know, I was referring to the +ve 2 billions. I could say a case 
> where Id generation could be set to start from a fairly large value and 
> so are the Id sequence increments. One could argue this is an edge case 
> ;-).

Can you please come up with a realistic use case where IDs would start 
on something other than 0 or 1? The database is controlled by Continuum 
and is an internal thing which we have complete control over.

> IMHO the version change to 1.1 is a fair indication that the API might 
> have changed. Having said that, I will go with whatever most of us think 
> sounds practical :-)

The only thing you can do is to add stuff, not break existing code.

--
Trygve

Mime
View raw message