Re: Review Request 49432: porting changes of AMBARI-16980 to hive2

2016-06-30 Thread Pallav Kulshreshtha

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


Ship it!




Ship It!

- Pallav Kulshreshtha


On June 30, 2016, 10:56 a.m., Nitiraj Rathore wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49432/
> ---
> 
> (Updated June 30, 2016, 10:56 a.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav 
> Kulshreshtha, Rohit Choudhary, and Ashwin Rajeev.
> 
> 
> Bugs: AMBARI-17484
> https://issues.apache.org/jira/browse/AMBARI-17484
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> changes include the history tab performance improvement
> 
> 
> Diffs
> -
> 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/persistence/utils/ItemNotFound.java
>  ad2adce 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/Aggregator.java
>  f184150 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/JobService.java
>  3bc396d 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/atsJobs/ATSParser.java
>  e465276 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/atsJobs/ATSRequestsDelegate.java
>  ac8cd22 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/atsJobs/ATSRequestsDelegateImpl.java
>  67497fd 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/atsJobs/IATSParser.java
>  e545c50 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/viewJobs/Job.java
>  816e77a 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/viewJobs/JobControllerImpl.java
>  e94d727 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/viewJobs/JobImpl.java
>  b71e2f7 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/viewJobs/JobInfo.java
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/components/number-range-widget.js
>  5d62b59 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/controllers/history.js
>  8c4ed2f 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/initializers/i18n.js
>  d2f6aaf 
>   contrib/views/hive-next/src/main/resources/ui/hive-web/app/models/job.js 
> 9079b5a 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/routes/history.js 
> 0aa3d41 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/services/history.js
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/templates/history.hbs
>  052498e 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/utils/constants.js 
> e40e447 
> 
> Diff: https://reviews.apache.org/r/49432/diff/
> 
> 
> Testing
> ---
> 
> manual testing done.
> 
> 
> Thanks,
> 
> Nitiraj Rathore
> 
>



Review Request 49432: porting changes of AMBARI-16980 to hive2

2016-06-30 Thread Nitiraj Rathore

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

Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav Kulshreshtha, 
Rohit Choudhary, and Ashwin Rajeev.


Bugs: AMBARI-17484
https://issues.apache.org/jira/browse/AMBARI-17484


Repository: ambari


Description
---

changes include the history tab performance improvement


Diffs
-

  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/persistence/utils/ItemNotFound.java
 ad2adce 
  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/Aggregator.java
 f184150 
  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/JobService.java
 3bc396d 
  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/atsJobs/ATSParser.java
 e465276 
  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/atsJobs/ATSRequestsDelegate.java
 ac8cd22 
  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/atsJobs/ATSRequestsDelegateImpl.java
 67497fd 
  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/atsJobs/IATSParser.java
 e545c50 
  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/viewJobs/Job.java
 816e77a 
  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/viewJobs/JobControllerImpl.java
 e94d727 
  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/viewJobs/JobImpl.java
 b71e2f7 
  
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/viewJobs/JobInfo.java
 PRE-CREATION 
  
contrib/views/hive-next/src/main/resources/ui/hive-web/app/components/number-range-widget.js
 5d62b59 
  
contrib/views/hive-next/src/main/resources/ui/hive-web/app/controllers/history.js
 8c4ed2f 
  
contrib/views/hive-next/src/main/resources/ui/hive-web/app/initializers/i18n.js 
d2f6aaf 
  contrib/views/hive-next/src/main/resources/ui/hive-web/app/models/job.js 
9079b5a 
  contrib/views/hive-next/src/main/resources/ui/hive-web/app/routes/history.js 
0aa3d41 
  
contrib/views/hive-next/src/main/resources/ui/hive-web/app/services/history.js 
PRE-CREATION 
  
contrib/views/hive-next/src/main/resources/ui/hive-web/app/templates/history.hbs
 052498e 
  contrib/views/hive-next/src/main/resources/ui/hive-web/app/utils/constants.js 
e40e447 

Diff: https://reviews.apache.org/r/49432/diff/


Testing
---

manual testing done.


Thanks,

Nitiraj Rathore