Ayende @ Rahien

It's a girl

A pull request with all of the taxes already paid, gimme that again!

I recently merged a pull request from Barry Hagan. The actual pull request is pretty boring, to tell you the truth, exposing a Lucene feature through the RavenDB API.

What really impressed me was how complete the pull request was.

What do I mean by complete?

A feature is not just supporting it in the engine. For this particular pull request, Barry have done:

  • Supported this in RavenDB Database Core.
  • The HTTP API.
  • The C# Client API.
  • The strongly typed C# Client API.
  • Included support for dynamic indexes.
  • Updated the query optimizer.
  • Exposed this feature in the UI.

Basically, the only things that I had to do were git pull and then review the code.

Very nicely done.


Posted By: Ayende Rahien

Published at

Originally posted at


02/19/2013 03:28 PM by

How does that work? People pay for RavenDB and still add features to it? And then have to pay for those features? I don't understand.

Ayende Rahien
02/19/2013 03:31 PM by
Ayende Rahien

Jiggaboo, This probably explains this best: http://twitter.com/randompunter/status/303499429313392642

02/19/2013 09:38 PM by

But when it comes to new features why not add it in you own solution and sell? You sir may have found ideal business! :)

Ayende Rahien
02/19/2013 10:02 PM by
Ayende Rahien

Jiggaboo, Because a new feature on its own is pretty useless. It is the aggregation that make things useful

Tim Murphy
02/19/2013 10:52 PM by
Tim Murphy

What about some tests to make it the perfect pull request :-)

Daniel Schilling
02/20/2013 07:32 PM by
Daniel Schilling

Looks to me like tests were included. Way to go, man!

02/20/2013 10:26 PM by

Nobody dares to send you a sloppy pull request. I wouldn't be surprised if some significant amount of work was put in that pull request, just to meet your standards :)

Comments have been closed on this topic.