Re: Review Request 36210: DRILL-3341: Move OperatorWrapper list and FragmentWrapper list creation to ProfileWrapper ctor

2015-07-20 Thread Jason Altekruse
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36210/#review92328 --- Ship it! Ship It! - Jason Altekruse On July 13, 2015, 11:08 p.m.

Re: Review Request 36210: DRILL-3341: Move OperatorWrapper list and FragmentWrapper list creation to ProfileWrapper ctor

2015-07-13 Thread Sudheesh Katkam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36210/ --- (Updated July 13, 2015, 11:08 p.m.) Review request for drill and Jason Altekrus

Re: Review Request 36210: DRILL-3341: Move OperatorWrapper list and FragmentWrapper list creation to ProfileWrapper ctor

2015-07-13 Thread Sudheesh Katkam
> On July 9, 2015, 2:59 p.m., Jason Altekruse wrote: > > exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java, > > line 105 > > > > > > Can we just change the name? Profiles d

Re: Review Request 36210: DRILL-3341: Move OperatorWrapper list and FragmentWrapper list creation to ProfileWrapper ctor

2015-07-09 Thread Jason Altekruse
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36210/#review90544 --- exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/prof

Review Request 36210: DRILL-3341: Move OperatorWrapper list and FragmentWrapper list creation to ProfileWrapper ctor

2015-07-06 Thread Sudheesh Katkam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36210/ --- Review request for drill and Jason Altekruse. Bugs: DRILL-3341 https://issu