Hi Thomas, Here is a new patch that takes account of your advice:
https://github.com/nicolas-f/H2-Mirror/compare/h2-fork-r5473 https://github.com/nicolas-f/H2-Mirror/compare/h2-fork-r5473.patch https://github.com/nicolas-f/H2-Mirror/compare/h2-fork-r5473.diff Thanks for your help -Nicolas Atelier SIG, IRSTV FR CNRS 2488 Le jeudi 30 janvier 2014 08:36:51 UTC+1, Thomas Mueller a écrit : > > Hi, > > The patch is good, thanks a lot! The only change I have in mind (but I can > do that) is to change the names of the interfaces. Currently you have: > > AggregateAlias (new base interface) > AggregateFunction extends AggregateAlias (the current interface, > deprecated) > AggregateTypeFunction extends AggregateAlias (new interface) > > Maybe it's easier to not use a base interface (not use AggregateAlias), > but deprecate the old interface (AggregateFunction), and create a new > interface and simply call that one "Aggregate". An implementation could > support both interfaces, in which case H2 would use "Aggregate" (the new > version). > > What do you think? > > Regards, > Thomas > -- You received this message because you are subscribed to the Google Groups "H2 Database" group. To unsubscribe from this group and stop receiving emails from it, send an email to h2-database+unsubscr...@googlegroups.com. To post to this group, send email to h2-database@googlegroups.com. Visit this group at http://groups.google.com/group/h2-database. For more options, visit https://groups.google.com/groups/opt_out.
