LGTM

On Tue, May 11, 2010 at 8:49 AM, <[email protected]> wrote:

> Reviewers: Ray Ryan,
>
> Description:
> Added a countExpenses method to the Expenses entity.
>
> Patch by: amitmanjhi
> Review by: rjrjr (TBR)
>
>
> Please review this at http://gwt-code-reviews.appspot.com/507801/show
>
> Affected files:
>  M
> trunk/bikeshed/src/com/google/gwt/sample/expenses/server/domain/Expense.java
>
>
> Index:
> trunk/bikeshed/src/com/google/gwt/sample/expenses/server/domain/Expense.java
> ===================================================================
> ---
> trunk/bikeshed/src/com/google/gwt/sample/expenses/server/domain/Expense.java
>        (revision 8045)
> +++
> trunk/bikeshed/src/com/google/gwt/sample/expenses/server/domain/Expense.java
>        (working copy)
> @@ -34,6 +34,15 @@
>  @Entity
>  public class Expense {
>
> +  public static long countExpenses() {
> +    EntityManager em = entityManager();
> +    try {
> +      return ((Number) em.createQuery("select count(o) from Expense
> o").getSingleResult()).longValue();
> +    } finally {
> +      em.close();
> +    }
> +  }
> +
>   public static final EntityManager entityManager() {
>     return EMF.get().createEntityManager();
>   }
>
>
>

-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to