Looks simple and cool, will add that one .

Thanks,

Satish


On 02/04/2013 03:38 PM, Bill Graham wrote:
>
> In hadoop-connector/com/vertica/hadoop/VerticaConfiguration.java:
>
> >  @@ -273,6 +304,7 @@ public Connection getConnection(boolean output) throws IOException,
> >   		String database = conf.get(DATABASE_PROP);
> >   		String port = conf.get(PORT_PROP);
> >   		String debug = conf.get(DEBUG_PROP);
> >  +		String logpath = conf.get(DEBUG_LOGPATH_PROP);
> >
> >
> >   		if((debug == null) || (debug == "")){
>
> You can get rid of this block by doing conf.get(DEBUG_PROP, 
> DEFAULT_DEBUG_PROP) above instead.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/vertica/Vertica-Hadoop-Connector/pull/4/files#r2882652>. 
>
>