jcsherin commented on PR #13040: URL: https://github.com/apache/datafusion/pull/13040#issuecomment-2435680521
> @jcsherin Oops ! I accidentally worked on `ntile` udwf, I was supposed to work on `nth` udwf. Also the branch name is wrong : [jatin510:feature/12649-udwf-nth-value](https://github.com/jatin510/datafusion/tree/feature/12649-udwf-nth-value) > > Should I delete this remote branch and update the branch name and create a PR again ! It is alright, that is not a problem at all 😂. You just need to edit the `Closes #` to refer instead to the `ntile` issue. - #12694 The `nth_value` conversion is also a good first issue which I hope somebody who wishes to learn DataFusion will pick up soon. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
