----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49379/ -----------------------------------------------------------
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