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


Ship it!




Since these are pre approved changes.

- Ashwin Rajeev


On June 29, 2016, 2:46 p.m., Nitiraj Rathore wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49379/
> -----------------------------------------------------------
> 
> (Updated June 29, 2016, 2:46 p.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav 
> Kulshreshtha, Rohit Choudhary, and Ashwin Rajeev.
> 
> 
> Bugs: AMBARI-17480
>     https://issues.apache.org/jira/browse/AMBARI-17480
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> port following change in hive2
> AMBARI-16266
> AMBARI-16424
> AMBARI-16425
> AMBARI-16422
> AMBARI-16726
> AMBARI-16675
> AMBARI-16429
> AMBARI-16242
> AMBARI-16418
> AMBARI-16428
> 
> 
> Diffs
> -----
> 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/persistence/DataStoreStorage.java
>  9adf129 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/Aggregator.java
>  4293b1b 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/jobs/JobService.java
>  975036e 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/uploads/UploadFromHdfsInput.java
>  7a7f77c 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/uploads/UploadService.java
>  59e969d 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/uploads/parsers/DataParser.java
>  6edc97c 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/uploads/parsers/IParser.java
>  401ef48 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/uploads/parsers/ParseUtils.java
>  fefacce 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/uploads/parsers/Parser.java
>  79c5482 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/uploads/parsers/csv/CSVParser.java
>  74cc060 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/uploads/parsers/json/JSONParser.java
>  772b6fd 
>   
> contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/uploads/parsers/xml/XMLParser.java
>  51671e5 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/adapters/file-upload.js
>  1bd8eee 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/adapters/upload-table.js
>  ef4df43 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/components/file-upload.js
>  1cd05ae 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/components/validated-text-field.js
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/controllers/index.js
>  4f3261c 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/controllers/upload-table.js
>  cb267a0 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/initializers/i18n.js
>  b3630c1 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/routes/history.js 
> 848bee8 
>   contrib/views/hive-next/src/main/resources/ui/hive-web/app/services/job.js 
> 6cb4170 
>   contrib/views/hive-next/src/main/resources/ui/hive-web/app/styles/app.scss 
> 8f60595 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/templates/components/validated-text-field.hbs
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/main/resources/ui/hive-web/app/templates/upload-table.hbs
>  b7d9011 
>   
> contrib/views/hive-next/src/test/java/org/apache/ambari/view/hive2/resources/upload/CSVParserTest.java
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/test/java/org/apache/ambari/view/hive2/resources/upload/DataParserCSVTest.java
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/test/java/org/apache/ambari/view/hive2/resources/upload/DataParserJSONTest.java
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/test/java/org/apache/ambari/view/hive2/resources/upload/DataParserXMLTest.java
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/test/java/org/apache/ambari/view/hive2/resources/upload/JsonParserTest.java
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/test/java/org/apache/ambari/view/hive2/resources/upload/QueryGeneratorTest.java
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/test/java/org/apache/ambari/view/hive2/resources/upload/TableDataReaderTest.java
>  PRE-CREATION 
>   
> contrib/views/hive-next/src/test/java/org/apache/ambari/view/hive2/resources/upload/XMLParserTest.java
>  PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/49379/diff/
> 
> 
> Testing
> -------
> 
> added test cases and also manual testing where required.
> 
> 
> Thanks,
> 
> Nitiraj Rathore
> 
>

Reply via email to