On 21.12.2014 17:07, Michael Michaud wrote:
> Hey Ede,
> 
> You made a big change on AttributeType class last thursday,
> It's a very interesting change with a lot of potential, but my
> concern is how much code it will break, and how much time
> it will take to upgrade all the code depending on this class.

valid concern
 
> New data types you inroduced are already available in Edit
> Schema plugin, but many drivers/plugin are not ready to
> manage these new types.

that was my intention and we have to start somewhere. the commit as such does 
not break a thing. when people start using the new attribute types and trun 
into errors we can start fixing them.

> From my point of view, this is the kind of change which may
> justify to create a new branch 

not again ;).. we are simply too few devs to maintain anymore than one proper 
branch

>as it may break current code

which code do you have in mind?

> for an undetermined laps of time.

shouldn't it break nothing? the "old" attribute types will continue to work as 
good as before. readers will probably still use the old datatypes and _only_ if 
the user switches the change will become effective, non?

> 
> What do you think ? Is it an intentional commit ?

yes. a first step, nothing more, nothing less.

> Are you aware of all the dependencies ?

the topic comes up from time to time, but we shy away from it. so why not start?

do you have an issue at hand, that breaks OJ's general usability because of 
this commit?

..ede


------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to