[ 
https://issues.apache.org/jira/browse/DERBY-2998?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12572781#action_12572781
 ] 

Daniel John Debrunner commented on DERBY-2998:
----------------------------------------------

AB> ... with patch 17. I'm not sure to what extent these need to be resolved 
before code can be committed, as they (appear to) only occur when ROW_NUMBER() 
is in use and thus shouldn't affect existing apps. But I'm not the one to say 
for sure one way or the other...

On the contrary, you *are* one of the ones to say if the patch should be 
committed. There is no one authority, just a set of committers. If any 
committer has confidence in the patch then she can commit it (and any other 
committer can veto it with a valid technical reason if she thinks it should not 
be committed).

With a patch reaching its 17th iteration I think one has to consider if it 
should be committed before too much of the effort is spent keeping the patch up 
to date with a moving codeline.

Is it better to keep trying to get a perfect patch (noting that the Derby 
project has ~450 open bugs so the base code is not perfect)?

Or is it better to get the code committed in some basic working state (has 
tests, doesn't cause existing tests to fail) so that others can try it out 
*and* ongoing changes don't cause regressions in the functionality.

And of course it's all done through svn, so reverting any patch is always an 
option.

> Add support for ROW_NUMBER() window function
> --------------------------------------------
>
>                 Key: DERBY-2998
>                 URL: https://issues.apache.org/jira/browse/DERBY-2998
>             Project: Derby
>          Issue Type: Sub-task
>          Components: SQL
>            Reporter: Thomas Nielsen
>            Assignee: Thomas Nielsen
>            Priority: Minor
>         Attachments: d2998-10.diff, d2998-10.stat, d2998-11.diff, 
> d2998-12.diff, d2998-12.stat, d2998-13.diff, d2998-13.stat, d2998-14.diff, 
> d2998-14.stat, d2998-15.diff, d2998-15.stat, d2998-16.diff, d2998-16.stat, 
> d2998-17.diff, d2998-17.stat, d2998-4.diff, d2998-4.stat, d2998-5.diff, 
> d2998-5.stat, d2998-6.diff, d2998-6.stat, d2998-7.diff, d2998-7.stat, 
> d2998-8.diff, d2998-8.stat, d2998-9-derby.log, d2998-9.diff, d2998-9.stat, 
> d2998-doc-1.diff, d2998-doc-1.stat, d2998-test.diff, d2998-test.stat, 
> d2998-test2.diff, d2998-test2.stat, d2998-test3.diff, d2998-test3.stat, 
> d2998-test4.diff, d2998-test4.stat, d2998-test6.diff, d2998-test7.diff
>
>
> As part of implementing the overall OLAP Operations features of SQL 
> (DERBY-581), implement the ROW_NUMBER() window function.
> More information about this feature is available at 
> http://wiki.apache.org/db-derby/OLAPRowNumber

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to