Hi James: The attached patch is good for me, thanks for your review.
On Wed, Apr 10, 2013 at 4:58 AM, James Peach <jpe...@apache.org> wrote: > Hi Yunkai, > > Leif and I reviewed the TS-1801 changes today. It looks good, though we > have one suggestion in NodeStatEval(), where there seems to be a > possibility that the result_type would not be set correctly. Do you think > that the attached patch is correct? > > > > > J > > > > -- Yunkai Zhang Work at Taobao
