falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Narayan Periwal" <narayan.peri...@inmobi.com>
Subject Re: Review Request 38710: Adding "-debug" option to Falcon CLI for debug statements to stdout.
Date Fri, 25 Sep 2015 06:51:38 GMT


> On Sept. 25, 2015, 5:43 a.m., Peeyush Bishnoi wrote:
> > client/src/main/java/org/apache/falcon/cli/FalconCLI.java, line 253
> > <https://reviews.apache.org/r/38710/diff/1/?file=1083841#file1083841line253>
> >
> >     Instead of Exception ex, can we use NumberFormatException, as parseInt throws
NumberFormatException. See if it is possible.

Making debug from String to boolean as suggested by boolean, so now we can continue with Exception
ex


- Narayan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38710/#review100533
-----------------------------------------------------------


On Sept. 24, 2015, 8:05 a.m., Narayan Periwal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38710/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2015, 8:05 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> With this feature, users can specify "-debug" in CLI arguments to get the debug statements
in Stdout. 
> eg. falcon entity -type feed -name in-feed.xml -submit -debug
> Also, users can export the env variable FALCON_DEBUG to a non-zero value to get the debug
statements on stdout.
> Starting right now with getting basic debug output such as the Http connection URL to
the server, this can be later extended for printing more debug statements at various stages.
> 
> 
> Diffs
> -----
> 
>   client/src/main/java/org/apache/falcon/cli/FalconCLI.java c914649 
>   client/src/main/java/org/apache/falcon/client/FalconClient.java 981559b 
> 
> Diff: https://reviews.apache.org/r/38710/diff/
> 
> 
> Testing
> -------
> 
> Done
> 
> 
> Thanks,
> 
> Narayan Periwal
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message