---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56379/#review165086
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On Feb. 10, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5/#review161841
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On Jan. 16, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52688/#review153680
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On Oct. 19, 2016
src/main/resources/ui/app/controllers/queuesconf.js
(line 252)
<https://reviews.apache.org/r/52688/#comment221260>
Is this conditional operator required?
- Sreenath Somarajapuram
On Oct. 10, 2016, 12:53 p.m., Akhil PB wrote:
>
> --
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51647/#review150829
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On Sept. 28
> On Sept. 23, 2016, 10:19 a.m., Sreenath Somarajapuram wrote:
> > contrib/views/capacity-scheduler/src/main/resources/ui/bower.json, line 16
> > <https://reviews.apache.org/r/51647/diff/1/?file=1491590#file1491590line16>
> >
> > Package versiosn m
<https://reviews.apache.org/r/51647/#comment218038>
PLease check where this action is called from.
contrib/views/capacity-scheduler/src/main/resources/ui/bower.json (line 16)
<https://reviews.apache.org/r/51647/#comment218031>
Package versiosn must be fixed.
- Sreenath Somar
emerick/jsdifflib.git#ce74529f9d51943db440dbd7c15d515597cda2e1";
- Sreenath Somarajapuram
On Sept. 6, 2016, 9:26 a.m., Akhil PB wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
&
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51452/#review147561
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On Sept. 1, 2016
ent214581>
Guess it might be better to put all finds into a promise array.
- Sreenath Somarajapuram
On Aug. 26, 2016, 7:09 a.m., Akhil PB wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://re
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51453/#review146932
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On Aug. 26, 2016
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49755/#review146662
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On Aug. 23, 2016
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51169/#review145969
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On Aug. 17, 2016
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50070/#review142361
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On July 15, 2016
ent207038>
Commented code.
contrib/views/capacity-scheduler/src/main/resources/ui/app/components/editLabelCapacity.js
(line 36)
<https://reviews.apache.org/r/49755/#comment207036>
Commented code.
- Sreenath Somarajapuram
On July 7, 2016, 1:10 p.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48487/#review140616
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On June 27, 2016
> On June 22, 2016, 10:12 a.m., Sreenath Somarajapuram wrote:
> > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/versionsPanel.hbs,
> > line 26
> > <https://reviews.apache.org/r/48487/diff/1/?file=1412307#file1412307line26>
> >
> &g
---
Manual testing.
Verified the presence of debug and error logs in
/var/log/ambari-server/tez-view/tez-view.log
Thanks,
Sreenath Somarajapuram
the presence of debug and error logs in
/var/log/ambari-server/tez-view/tez-view.log
Thanks,
Sreenath Somarajapuram
in
/var/log/ambari-server/tez-view/tez-view.log
Thanks,
Sreenath Somarajapuram
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48686/#review139293
-------
On June 23, 2016, 8:04 p.m., Sreenath Somarajapuram wrote:
>
>
iff: https://reviews.apache.org/r/48686/diff/
Testing
---
Manual testing.
Verified the presence of debug and error logs in
/var/log/ambari-server/tez-view/tez-view.log
Thanks,
Sreenath Somarajapuram
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49141/#review139227
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On June 23, 2016
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48487/#review139051
---
- Sreenath Somarajapuram
On June 9, 2016, 12:05 p.m., Akhil PB
> On June 22, 2016, 10:12 a.m., Sreenath Somarajapuram wrote:
> > contrib/views/capacity-scheduler/src/main/resources/ui/app/templates/versionsPanel.hbs,
> > line 26
> > <https://reviews.apache.org/r/48487/diff/1/?file=1412307#file1412307line26>
> >
> &g
store passed?
- Sreenath Somarajapuram
On June 9, 2016, 12:05 p.m., Akhil PB wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://re
Where is the versions-table-wrap class defined?
Wouldnt it cause the views/queues.js code to break?
- Sreenath Somarajapuram
On June 9, 2016, 12:05 p.m., Akhil PB wrote:
>
> ---
> This is an automatically generated e-mai
> On June 22, 2016, 8:50 a.m., Sreenath Somarajapuram wrote:
> > contrib/views/capacity-scheduler/src/main/resources/ui/app/serializers.js,
> > line 164
> > <https://reviews.apache.org/r/48487/diff/1/?file=1412286#file1412286line164>
> >
> > Why ar
/serializers.js (line
164)
<https://reviews.apache.org/r/48487/#comment204205>
Why are we having the + after ?
- Sreenath Somarajapuram
On June 9, 2016, 12:05 p.m., Akhil PB wrote:
>
> ---
> This is an automatically gener
/apache/ambari/view/tez/rest/BaseRedirectionResource.java
97ee01d
Diff: https://reviews.apache.org/r/48686/diff/
Testing
---
Manual testing.
Verified the presence of debug and error logs in
/var/log/ambari-server/tez-view/tez-view.log
Thanks,
Sreenath Somarajapuram
/BaseRedirectionResource.java
97ee01d
Diff: https://reviews.apache.org/r/48686/diff/
Testing
---
Manual testing.
Verified the presence of debug and error logs in
/var/log/ambari-server/tez-view/tez-view.log
Thanks,
Sreenath Somarajapuram
Diff: https://reviews.apache.org/r/48686/diff/
Testing
---
Manual testing.
Verified the presence of debug and error logs in
/var/log/ambari-server/tez-view/tez-view.log
Thanks,
Sreenath Somarajapuram
/r/48582/diff/
Testing
---
Manually tested
Thanks,
Sreenath Somarajapuram
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48398/#review136986
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On June 10, 2016
e tofixed
http://www.w3schools.com/jsref/jsref_tofixed.asp
- Sreenath Somarajapuram
On June 8, 2016, 6:42 a.m., Akhil PB wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> http
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47814/#review136447
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On May 25, 2016
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47643/#review134326
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On May 23, 2016
/templates/capsched/queuesconf/editqueue.hbs
(line 50)
<https://reviews.apache.org/r/47643/#comment198832>
This need to be corrected.
- Sreenath Somarajapuram
On May 20, 2016, 9:57 a.m., Akhil PB wrote:
>
> ---
> This is a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46770/#review132651
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On May 9, 2016
-view.js 4b7d94d
Diff: https://reviews.apache.org/r/47097/diff/
Testing
---
Thanks,
Sreenath Somarajapuram
ent195837>
Can we get this removed?
- Sreenath Somarajapuram
On April 28, 2016, 11:43 a.m., Akhil PB wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://re
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44704/#review130753
---
Ship it!
Ship It!
- Sreenath Somarajapuram
On April 27
/controllers/queuesconf.js
(line 138)
<https://reviews.apache.org/r/44704/#comment194613>
Where is operation defined?
- Sreenath Somarajapuram
On April 26, 2016, 10:43 a.m., Akhil PB wrote:
>
> ---
> This is an automatica
against Tez UI 1 & 2.
Thanks,
Sreenath Somarajapuram
/controllers/queuesconf.js
(line 76)
<https://reviews.apache.org/r/44704/#comment189899>
Commented code.
- Sreenath Somarajapuram
On March 11, 2016, 10:41 a.m., Akhil PB wrote:
>
> ---
> This is an automatically generated e
views/editqueue.js
(line 28)
<https://reviews.apache.org/r/44704/#comment189893>
Better to off these listeners on willDestroy.
- Sreenath Somarajapuram
On March 11, 2016, 10:41 a.m., Akhil PB wrote:
>
> ---
> This is an a
ib/views/capacity-scheduler/src/main/resources/ui/app/templates/capsched.hbs
(line 28)
<https://reviews.apache.org/r/44704/#comment189888>
Whay are we having an anchor tags inside link-to? And what is the purpose
fo href="view.href".
- Sreenath Somarajap
47 matches
Mail list logo