> On April 12, 2017, 5:35 a.m., Amareshwari Sriramadasu wrote:
> > lens-server/src/test/java/org/apache/lens/server/metastore/TestMetastoreService.java
> > Lines 1958 (patched)
> > <https://reviews.apache.org/r/58338/diff/2/?file=1687937#file1687937line1958>
> >
> >     Can you update test for verfiying the error as well?

Reopening this. Can you update test verify error scenario as well?


- Amareshwari


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


On April 12, 2017, 2:25 p.m., Lavkesh Lahngir wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58338/
> -----------------------------------------------------------
> 
> (Updated April 12, 2017, 2:25 p.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> Bugs: LENS-1410
>     https://issues.apache.org/jira/browse/LENS-1410
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> A string filer for getting for partition api must tbe provided
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/resources/lens-errors.conf 29e24cf 
>   lens-cli/src/main/java/org/apache/lens/cli/commands/LensFactCommands.java 
> 9ea887a 
>   lens-cli/src/test/java/org/apache/lens/cli/TestLensFactCommands.java 
> f176078 
>   lens-cube/src/main/java/org/apache/lens/cube/error/LensCubeErrorCode.java 
> 571b481 
>   
> lens-server/src/main/java/org/apache/lens/server/metastore/MetastoreResource.java
>  9d823da 
>   
> lens-server/src/test/java/org/apache/lens/server/metastore/TestMetastoreService.java
>  f6f6e77 
> 
> 
> Diff: https://reviews.apache.org/r/58338/diff/3/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Lavkesh Lahngir
> 
>

Reply via email to