On 31 July 2013 13:04, Gunnar Morling <gunnar(a)hibernate.org> wrote:
2013/7/31 Hardy Ferentschik <hardy(a)hibernate.org>
> On 31 Jan 2013, at 1:38 PM, Sanne Grinovero <sanne(a)hibernate.org> wrote:
> > Hi Hardy,
> > sorry I screwed up both during review and setting up ci.hibernate.org
> > : the code in Maven module _hibernate-search-performance_ isn't
> > compiling anymore; could you extend your refactoring to this module
> > please?
> :-( I can have a look tomorrow
> > This slipped in as the profile is disabled by default; I'm fixing the
> > pull requests checker to verify this profile as well. I suspect I
> > wanted to keep the PR checker quick and lean, but it doesn't take too
> > long now.
> And this is exactly why I always say that all modules should build by
> (with options for the performance minded to disable certain
> Also getting feedback from the pull request checker is ways too late. By
> the time
> I get feedback from there I have moved on and work on another branch.
> At least the code parts needs to be all compiled on each build.
> But I guess I stand alone with this opinion :-(
No, you don't. Take OGM for example, there we just enabled the MongoDB
modules to run as part of the default build.
It's a tradeoff, it's not the end of the world if this module doesn't
compile for day.
On the other hand I run the build dozens of times a day, quite happy
it takes just 2 minutes, which is not possible with the performance
Another approach would be to move those tests into a different
repository; but technically I wish I had merged my other tests sooner
as now they are outdated and need maintenance.
I don't think we should automate this all to make our life too easy:
engineers!=mindless chickens. it was my mistake, should just think
better of the consequences of code change and I think we can generally
do that, and be forgiving for the occasional mistake as it saves lots
The MongoDB example is different; I have always run those tests, I
just had to put a 10 minutes investment to install MongoDB and setup
the variables. The improvement is welcome but I think mostly meant for
eventual occasional contributors not interested in the MongoDB
> hibernate-dev mailing list