Actually error reporting needs to be improved in the entire config system - 
which is a larger undertaking. I don't think the release needs to wait for 
that. Let's commit FLUME-1743 and then roll another (hopefully final) RC.  

Thanks,
Hari

-- 
Hari Shreedharan


On Wednesday, November 28, 2012 at 1:58 PM, Brock Noland wrote:

> I think 1743 should also make this obvious to the user. Our error reporting 
> here needs to be be improved. 
> 
> -- 
> Brock Noland
> Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
> 
> 
> On Wednesday, November 28, 2012 at 3:45 PM, Will McQueen wrote:
> 
> > Hi Brock,
> > 
> > Initially I saw some issue with mismatched source file, but I no longer see
> > that after doing a 'mvn clean install' instead of just 'mvn install'. Now,
> > I see the same problems as Roshan and Alexander: in both trunk and
> > flume-1.3.0, the multiport syslog tcp source is failing to configure
> > regardless of whether we specify the FQCN or the short alias.
> > 
> > With the patch that Hari submitted to FLUME-1743, I no longer see the issue
> > in trunk and flume-1.3.0 branches. Given that there's neither the FQCN nor
> > the alias work without the patch, I would consider this to be a blocker for
> > 1.3.0 release so I've upgraded the FLUME-1743's priority to "blocker".
> > 
> > Cheers,
> > Will
> > 
> > 
> > On Wed, Nov 28, 2012 at 12:59 PM, Brock Noland <[email protected] 
> > (mailto:[email protected])> wrote:
> > 
> > > Hmm it looks like we are eating whatever exception is occurring in
> > > FlumeConfiguration.valdiateSources and
> > > then ComponentConfigurationFactory.create().
> > > 
> > > I'll look at this tomorrow unless someone else gets to it sooner.
> > > 
> > > Brock
> > > 
> > > On Wed, Nov 28, 2012 at 2:46 PM, Roshan Naik <[email protected] 
> > > (mailto:[email protected])>
> > > wrote:
> > > 
> > > > In rc5, both FQCN and short name gave same error.
> > > > 
> > > > -roshan
> > > 
> > > 
> > > 
> > > --
> > > Apache MRUnit - Unit testing MapReduce -
> > > http://incubator.apache.org/mrunit/
> > > 
> > 
> > 
> 
> 
> 


Reply via email to