Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 Search Hadoop and all its subprojects:

Switch to Threaded View
HBase >> mail # dev >> unit tests improvement


Copy link to this message
-
Re: unit tests improvement
On Thu, Nov 3, 2011 at 9:53 AM, N Keywal <[EMAIL PROTECTED]> wrote:

> > > 2) HBASE-4737 is for the split; as discussed two weeks ago. There is a
> > > split proposal. I will apply this split this week-end or monday, don't
> > > hesitate to provide a feedback before (or after if necessary :-). I
> > tested
> > > the parallelization within surefire for the small tests, it seems to
> work
> > > quite well.
> > >
> > > Don't forget that the idea is to run only small & medium on the
> developer
> > > machine by default. This will make the sub-tests-suite runs under 30
> > > minutes, but the selection is important if we don't want to kill the
> > > pre-patch machine with defects that could have been detected before.
> > >
> >
> > You thinking patch-build should not run the full suite?
> >
> >
> I was thinking about the hbase-book, chapter "13.7 submitting patch". Today
> it says; " Make sure unit tests pass locally before submitting the patch.".
> It could become something as: "Make sure unit tests pass locally before
> submitting the patch. For large or risky patches, run as well the
> integration/large tests suite before submitting".
>

+1

We might want to qualify 'large' as spanning X classes (4+?) so people have
something to go by. However, this doesn't need to be a hard and fast rule
as jenkins should be running the entire suite.

--
-------------------
Jesse Yates
240-888-2200
@jesse_yates
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB