Re: Review Request 48122: History tab takes long to populate when there is more entry in history table.

2016-06-02 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48122/#review136032 --- Ship it! Ship It! - Pallav Kulshreshtha On June 1, 2016, 1

Re: Review Request 48119: View Instance: Data Migration

2016-06-02 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48119/#review136033 --- Ship it! Ship It! - Pallav Kulshreshtha On June 1, 2016, 10

Re: Review Request 48276: View config-Custom views are not working after kerberization

2016-06-06 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48276/#review136398 --- Ship it! Ship It! - Pallav Kulshreshtha On June 6, 2016, 12

Re: Review Request 48274: Show better error message when VIEW USER can not access CAPACITY SCHEDULER View

2016-06-06 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48274/#review136397 --- Ship it! Ship It! - Pallav Kulshreshtha On June 6, 2016, 12

Re: Review Request 47814: Capacity Scheduler View - Save only, Save and Restart Rm and Delete queue integrations

2016-06-06 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47814/#review136396 --- Ship it! Ship It! - Pallav Kulshreshtha On May 25, 2016, 6

Review Request 48404: Add user home directory verification as part of Service Check

2016-06-08 Thread Pallav Kulshreshtha
1d848e3 contrib/views/pig/src/main/resources/ui/pig-web/app/templates/splash.hbs d635a91 contrib/views/pig/src/main/resources/ui/pig-web/app/translations.js 072b5c7 Diff: https://reviews.apache.org/r/48404/diff/ Testing --- manually tested Thanks, Pallav Kulshreshtha

Review Request 48279: Visual explain unit tests

2016-06-06 Thread Pallav Kulshreshtha
/unit/components/udf-tr-view-test.js PRE-CREATION contrib/views/hive/src/main/resources/ui/hive-web/tests/unit/controllers/udfs-test.js 5bd369e Diff: https://reviews.apache.org/r/48279/diff/ Testing --- ember test --server Thanks, Pallav Kulshreshtha

Re: Review Request 48404: Add user home directory verification as part of Service Check

2016-06-10 Thread Pallav Kulshreshtha
--- manually tested Thanks, Pallav Kulshreshtha

Re: Review Request 48480: View Config- views created for 2nd remote cluster points to 1st remote cluster

2016-06-10 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48480/#review136980 --- Ship it! Ship It! - Pallav Kulshreshtha On June 9, 2016, 8

Review Request 48027: Files View : Table row overlaps if group name is long

2016-05-30 Thread Pallav Kulshreshtha
/templates/components/file-row.hbs a631335 Diff: https://reviews.apache.org/r/48027/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 47950: Invalid syntax of Oracle DDL for cluster_handle in viewinstance

2016-05-27 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47950/#review135221 --- Ship it! Ship It! - Pallav Kulshreshtha On May 27, 2016, 10

Re: Review Request 47738: [Hive View] Non-visual explain gives a bunch of strangely clickable links

2016-05-26 Thread Pallav Kulshreshtha
3f8810f Diff: https://reviews.apache.org/r/47738/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 47738: [Hive View] Non-visual explain gives a bunch of strangely clickable links

2016-05-26 Thread Pallav Kulshreshtha
-- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47738/#review134574 ------- On May 23, 2016, 6:44 p.m., Pallav Kulshreshtha wrote: > > -

Review Request 48114: Views Config- UI shows incorrect labels

2016-06-01 Thread Pallav Kulshreshtha
/views/modals/DeregisterClusterModal.html PRE-CREATION ambari-admin/src/main/resources/ui/admin-web/app/views/modals/deregisterClusterModal.html e6c67ca Diff: https://reviews.apache.org/r/48114/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Review Request 48123: Update default ambari view read timeout to a larger value

2016-06-01 Thread Pallav Kulshreshtha
/org/apache/ambari/server/configuration/Configuration.java 900083f Diff: https://reviews.apache.org/r/48123/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Review Request 48069: REmote cluster with non-URI safe chars cause problems

2016-05-31 Thread Pallav Kulshreshtha
/admin-web/app/views/remoteClusters/remoteClusterPage.html 0f82360 Diff: https://reviews.apache.org/r/48069/diff/ Testing --- manually tested Thanks, Pallav Kulshreshtha

Re: Review Request 49141: Capacity Scheduler View: Javascript error causing new queue creation failed in Yarn Capacity Scheduler

2016-06-23 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49141/#review139232 --- Ship it! Ship It! - Pallav Kulshreshtha On June 23, 2016, 9

Review Request 49144: FilesView: Adopt front-end assets to support proxy

2016-06-23 Thread Pallav Kulshreshtha
/src/main/resources/ui/app/mixins/file-operation.js 6f4d070 contrib/views/files/src/main/resources/ui/ember-cli-build.js ab684c4 Diff: https://reviews.apache.org/r/49144/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 48964: Zeppelin: Kerberos configurations and latest interpreter config missing

2016-06-20 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48964/#review138745 --- Ship it! Ship It! - Pallav Kulshreshtha On June 20, 2016, 8

Review Request 49194: [Hive view] hiveserver service check fails for LDAP configured hive view

2016-06-24 Thread Pallav Kulshreshtha
PRE-CREATION contrib/views/hive/src/main/resources/ui/hive-web/app/utils/constants.js ec8352b Diff: https://reviews.apache.org/r/49194/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 49162: Update Zeppelin service definition

2016-06-24 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49162/#review139374 --- Ship it! Ship It! - Pallav Kulshreshtha On June 23, 2016, 7

Review Request 44611: file path spills out of notification panel

2016-03-09 Thread Pallav Kulshreshtha
trib/views/files/src/main/resources/ui/app/templates/messages/message.hbs e3eacec Diff: https://reviews.apache.org/r/44611/diff/ Testing --- manually tested. CSS changes. Thanks, Pallav Kulshreshtha

Review Request 44845: Manual re-execution is required after entering the password for LDAP

2016-03-15 Thread Pallav Kulshreshtha
iff: https://reviews.apache.org/r/44845/diff/ Testing --- manully tested, no need of adding any new unit test case. Thanks, Pallav Kulshreshtha

Review Request 44565: Hive view: Visual Explain does not work when there are multiple statements in the query editor

2016-03-09 Thread Pallav Kulshreshtha
nit test case needed. Thanks, Pallav Kulshreshtha

Re: Review Request 44773: Hive View : Upload Table : Tables are not deleted in case of exception.

2016-03-19 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44773/#review123976 --- Ship it! Ship It! - Pallav Kulshreshtha On March 14, 2016

Re: Review Request 44565: Hive view: Visual Explain does not work when there are multiple statements in the query editor

2016-03-19 Thread Pallav Kulshreshtha
4565/diff/ Testing --- manual testing done. No extra unit test case needed. Thanks, Pallav Kulshreshtha

Review Request 45866: File browser view should have some checks before starting similar to pig view

2016-04-07 Thread Pallav Kulshreshtha
/resources/ui/app/templates/splash.hbs PRE-CREATION Diff: https://reviews.apache.org/r/45866/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 45865: Ambari Views : Reverting the changes for separation of logs in ambari branch-2.2

2016-04-07 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45865/#review127587 --- Ship it! Ship It! - Pallav Kulshreshtha On April 7, 2016

Review Request 45864: Hive view should have some checks before starting similar to pig view

2016-04-07 Thread Pallav Kulshreshtha
/app/routes/splash.js PRE-CREATION contrib/views/hive/src/main/resources/ui/hive-web/app/templates/splash.hbs PRE-CREATION Diff: https://reviews.apache.org/r/45864/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Review Request 46082: Tittle of permissions window spilling out of the line.

2016-04-12 Thread Pallav Kulshreshtha
://reviews.apache.org/r/46082/diff/ Testing --- manually tested Thanks, Pallav Kulshreshtha

Re: Review Request 45805: "E090 NullPointerException" when executing Hive queries with tez

2016-04-06 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45805/#review127316 --- Ship it! Ship It! - Pallav Kulshreshtha On April 6, 2016, 7

Review Request 45379: Introduce "Copy Path to clipboard" feature for Files browser view UI

2016-03-28 Thread Pallav Kulshreshtha
/files.js 9fc11b6 contrib/views/files/src/main/resources/ui/app/templates/files.hbs 3e178bc contrib/views/files/src/main/resources/ui/package.json 8250489 Diff: https://reviews.apache.org/r/45379/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 45379: Introduce "Copy Path to clipboard" feature for Files browser view UI

2016-03-30 Thread Pallav Kulshreshtha
: https://reviews.apache.org/r/45379/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 46728: AMBARI-16133: Zeppelin to select only tested Interpreter

2016-04-28 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46728/#review130895 --- Ship it! Ship It! - Pallav Kulshreshtha On April 27, 2016

Review Request 46726: Date Filter editable in History Tab

2016-04-27 Thread Pallav Kulshreshtha
-range-widget.js 9e38786 Diff: https://reviews.apache.org/r/46726/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 47604: View URLs: UX edits #4

2016-05-20 Thread Pallav Kulshreshtha
/admin-web/app/views/urls/edit.html 9e0ef4c Diff: https://reviews.apache.org/r/47604/diff/ Testing --- manually tested Thanks, Pallav Kulshreshtha

Re: Review Request 47540: [Hive View]Unavailable Column Search should throw 'No Match' message

2016-05-20 Thread Pallav Kulshreshtha
--- manually tested Thanks, Pallav Kulshreshtha

Re: Review Request 47721: Editing local cluster name breaks all view instances

2016-05-23 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47721/#review134409 --- Ship it! Ship It! - Pallav Kulshreshtha On May 23, 2016, 10

Review Request 47738: [Hive View] Non-visual explain gives a bunch of strangely clickable links

2016-05-23 Thread Pallav Kulshreshtha
/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 47526: Hive View : History tab in hive view does not show recent jobs

2016-05-19 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47526/#review133918 --- Ship it! Ship It! - Pallav Kulshreshtha On May 18, 2016, 7

Re: Review Request 47525: Hive View : Upload table still shows file name after the upload is done

2016-05-19 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47525/#review133919 --- Ship it! Ship It! - Pallav Kulshreshtha On May 18, 2016, 6

Re: Review Request 47097: Tez View : Update namespace to use version info

2016-05-19 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47097/#review133932 --- Ship it! Ship It! - Pallav Kulshreshtha On May 8, 2016, 6

Re: Review Request 47537: View log files should pickup ambari-server log directory path

2016-05-19 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47537/#review133917 --- Ship it! Ship It! - Pallav Kulshreshtha On May 18, 2016, 3

Re: Review Request 47524: Hive view : For Upload Table, 'default' DB should be selected by default

2016-05-19 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47524/#review133921 --- Ship it! Ship It! - Pallav Kulshreshtha On May 18, 2016, 6

Review Request 47604: View URLs: UX edits #4

2016-05-19 Thread Pallav Kulshreshtha
://reviews.apache.org/r/47604/diff/ Testing --- manually tested Thanks, Pallav Kulshreshtha

Re: Review Request 47445: Remote Cluster: UX edits #1

2016-05-19 Thread Pallav Kulshreshtha
--- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 47643: Capacity Scheduler View - Edit and validate queue name

2016-05-24 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47643/#review134530 --- Ship it! Ship It! - Pallav Kulshreshtha On May 23, 2016, 5

Re: Review Request 47696: Upload Table- Tries to upload table, when no file is mentioned

2016-05-24 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47696/#review134555 --- Ship it! Ship It! - Pallav Kulshreshtha On May 24, 2016, 9

Re: Review Request 47460: Explain script on pig views encountered an IOExecption

2016-05-19 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47460/#review133999 --- Ship it! Ship It! - Pallav Kulshreshtha On May 17, 2016, 3

Review Request 47229: [UI Issue]Execution of empty pig script allowed from UI

2016-05-11 Thread Pallav Kulshreshtha
/scriptEdit.js 60a04ce contrib/views/pig/src/main/resources/ui/pig-web/app/translations.js 5bb8cf7 Diff: https://reviews.apache.org/r/47229/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Review Request 47224: [Hive View] - Save Default Settings should not permit empty parameter values

2016-05-11 Thread Pallav Kulshreshtha
setting while saving the settings. Diffs - contrib/views/hive/src/main/resources/ui/hive-web/app/services/settings.js b813bbf Diff: https://reviews.apache.org/r/47224/diff/ Testing --- manually tested Thanks, Pallav Kulshreshtha

Review Request 47236: [Hive View] EXPLAIN text is spilling out

2016-05-11 Thread Pallav Kulshreshtha
/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 47175: Hive View : Upload table : show UI validation errors while creating table.

2016-05-12 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47175/#review132883 --- Ship it! Ship It! - Pallav Kulshreshtha On May 10, 2016, 10

Re: Review Request 47178: Hive view : Upload Table throws exception when 'stored as' is set to 'INPUTFORMAT'

2016-05-12 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47178/#review132882 --- Ship it! Ship It! - Pallav Kulshreshtha On May 10, 2016, 11

Review Request 47290: [Hive view] Break in UI for Query Settings in Firefox

2016-05-12 Thread Pallav Kulshreshtha
://reviews.apache.org/r/47290/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 47124: Hive View : Upload Table- Error on uploading csv with 1 row

2016-05-16 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47124/#review133348 --- Ship it! Ship It! - Pallav Kulshreshtha On May 14, 2016, 8

Review Request 47540: [Hive View]Unavailable Column Search should throw 'No Match' message

2016-05-18 Thread Pallav Kulshreshtha
/views/hive/src/main/resources/ui/hive-web/app/initializers/i18n.js d2f6aaf contrib/views/hive/src/main/resources/ui/hive-web/app/templates/databases-search-results.hbs 007d9ca Diff: https://reviews.apache.org/r/47540/diff/ Testing --- manually tested Thanks, Pallav Kulshreshtha

Re: Review Request 47025: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Admin changes

2016-05-11 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47025/#review132611 --- Ship it! Ship It! - Pallav Kulshreshtha On May 9, 2016, 1

Re: Review Request 47229: [UI Issue]Execution of empty pig script allowed from UI

2016-05-12 Thread Pallav Kulshreshtha
/main/resources/ui/pig-web/app/controllers/scriptEdit.js 60a04ce contrib/views/pig/src/main/resources/ui/pig-web/app/translations.js 5bb8cf7 Diff: https://reviews.apache.org/r/47229/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 47025: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Admin changes

2016-05-05 Thread Pallav Kulshreshtha
/controllers/remoteClusters/RemoteClustersEditCtrl.js (line 117) <https://reviews.apache.org/r/47025/#comment195940> 'password' should be removed, as API does not need this param any more. - Pallav Kulshreshtha On May 5, 2016, 4:25 p.m., Gaurav Nagar

Re: Review Request 47025: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Admin changes

2016-05-05 Thread Pallav Kulshreshtha
/remoteClusters/modals/changePassword.html (line 28) <https://reviews.apache.org/r/47025/#comment195944> 'Username Required!' should come from i18-config file. - Pallav Kulshreshtha On May 5, 2016, 4:25 p.m., Gaurav Nagar

Re: Review Request 47025: Views: Simplify View Configuration for Remote Ambari Managed Cluster. Ambari Admin changes

2016-05-05 Thread Pallav Kulshreshtha
/remoteClusters/editRemoteClusterPage.html (line 40) <https://reviews.apache.org/r/47025/#comment195943> Add one more validation 'Special chars not allowed' same as create remote cluster page. - Pallav Kulshreshtha On May 5, 2016, 4:25 p.m., Gaurav Nagar

Re: Review Request 47124: Hive View : Upload Table- Error on uploading csv with 1 row

2016-05-10 Thread Pallav Kulshreshtha
/resources/upload/DataParserCSVTest.java (line 63) <https://reviews.apache.org/r/47124/#comment196583> Pls remove commented lines from the code. - Pallav Kulshreshtha On May 10, 2016, 4:41 a.m., Nitiraj Rathore

Re: Review Request 46210: Services status check labels and progress bar changes for Hive, Pig and Files view

2016-04-15 Thread Pallav Kulshreshtha
to change the code on HTML. - Pallav --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46210/#review128958 --- On April 14, 201

Review Request 46210: Services status check labels and progress bar changes for Hive, Pig and Files view

2016-04-14 Thread Pallav Kulshreshtha
/ui/pig-web/app/routes/splash.js 5223a5a contrib/views/pig/src/main/resources/ui/pig-web/app/templates/splash.hbs 7455a4f Diff: https://reviews.apache.org/r/46210/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 47592: AMBARI-16772 Zeppelin restart with Ambari fails - according to Ambari

2016-05-25 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47592/#review134748 --- Ship it! Ship It! - Pallav Kulshreshtha On May 25, 2016, 10

Re: Review Request 49946: Zeppelin service: add default kerberos config for shell interpreter

2016-07-12 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49946/#review141864 --- Ship it! Ship It! - Pallav Kulshreshtha On July 12, 2016, 6

Re: Review Request 50031: Error for Pig view after upgrade to 2.4.0.0 [PersistenceException: Can't initialize data store for view PIG]

2016-07-14 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50031/#review142245 --- Ship it! Ship It! - Pallav Kulshreshtha On July 14, 2016, 1

Re: Review Request 50328: Local and custom views are not working when swebhdfs and Namenode HA is enabled

2016-07-22 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50328/#review143195 --- Ship it! Ship It! - Pallav Kulshreshtha On July 22, 2016, 6

Re: Review Request 50792: "ItemNotFound" error on uploading table hive View

2016-08-04 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50792/#review144750 --- Ship it! Ship It! - Pallav Kulshreshtha On Aug. 4, 2016, 12

Re: Review Request 50070: Capacity Scheduler View - Fix for nodelabel api compatibility issue in HDP-2.5

2016-07-15 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50070/#review142365 --- Ship it! Ship It! - Pallav Kulshreshtha On July 15, 2016

Re: Review Request 50189: Getting error while executing query for adding udf jar

2016-07-19 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50189/#review142737 --- Ship it! Ship It! - Pallav Kulshreshtha On July 19, 2016

Re: Review Request 49750: Upload table using Hive view will fail if Hive database is created with Location option

2016-07-18 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49750/#review142548 --- Ship it! Ship It! - Pallav Kulshreshtha On July 7, 2016, 11

Re: Review Request 50129: Hive View : Clicking on History button in saved search does not show history page

2016-07-18 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50129/#review142550 --- Ship it! Ship It! - Pallav Kulshreshtha On July 18, 2016

Re: Review Request 49857: Spark Interpreter fails with "HiveException: org.apache.thrift.transport.TTransportException"

2016-07-11 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49857/#review141664 --- Ship it! Ship It! - Pallav Kulshreshtha On July 9, 2016, 6

Re: Review Request 49859: Zeppelin service: Dependencies for phoenix in JDBC interpreter are not configured by default

2016-07-11 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49859/#review141663 --- Ship it! Ship It! - Pallav Kulshreshtha On July 9, 2016, 6

Re: Review Request 49858: Zeppelin service: remove principal and keytab from interpreter settings when kerberos is disabled on a secure cluster

2016-07-11 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49858/#review141662 --- Ship it! Ship It! - Pallav Kulshreshtha On July 9, 2016, 6

Re: Review Request 50850: Pig view - Pig script creation fails

2016-08-05 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50850/#review144931 --- Ship it! Ship It! - Pallav Kulshreshtha On Aug. 5, 2016, 1

Re: Review Request 50850: Pig view - Pig script creation fails

2016-08-08 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50850/#review145101 --- Ship it! Ship It! - Pallav Kulshreshtha On Aug. 8, 2016, 2

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

Re: Review Request 49495: Upload Table - picks different datatype for same file, for different hive view version.

2016-07-01 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49495/#review140329 --- Ship it! Ship It! - Pallav Kulshreshtha On July 1, 2016, 7

Re: Review Request 49450: Port changes of AMBARI-17421 to hive-jdbc view

2016-07-01 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49450/#review140310 --- Ship it! Ship It! - Pallav Kulshreshtha On June 30, 2016, 1

Re: Review Request 49379: Hive2 view : port changes of following 10 bugs in hive2 view

2016-06-29 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49379/#review139998 --- Ship it! Ship It! - Pallav Kulshreshtha On June 29, 2016, 2

Review Request 49427: Hive2 view : port changes of following 8 UI bugs in hive2 view

2016-06-30 Thread Pallav Kulshreshtha
/components/udf-tr-view-test.js PRE-CREATION contrib/views/hive-next/src/main/resources/ui/hive-web/tests/unit/controllers/udfs-test.js 5bd369e Diff: https://reviews.apache.org/r/49427/diff/ Testing --- manually tested Thanks, Pallav Kulshreshtha

Re: Review Request 56168: AMBARI-19819. Styling changes and spelling fixes

2017-02-01 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56168/#review163939 --- Ship it! Ship It! - Pallav Kulshreshtha On Feb. 1, 2017, 1

Re: Review Request 56115: Hive2: Unreasonable response time from query exeuction in hive view

2017-01-31 Thread Pallav Kulshreshtha
/main/resources/ui/app/routes/queries/query.js 753b7f7 contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs 5cdc9b3 Diff: https://reviews.apache.org/r/56115/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Review Request 56115: Hive2: Unreasonable response time from query exeuction in hive view

2017-01-31 Thread Pallav Kulshreshtha
/query.js 753b7f7 contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs 5cdc9b3 Diff: https://reviews.apache.org/r/56115/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 56084: Hive2: Results are not cleared on "execute"

2017-01-31 Thread Pallav Kulshreshtha
-table.hbs 4fe6f43 contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs 5cdc9b3 Diff: https://reviews.apache.org/r/56084/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 56084: Hive2: Results are not cleared on "execute"

2017-01-31 Thread Pallav Kulshreshtha
-table.hbs 180c43d contrib/views/hive20/src/main/resources/ui/app/templates/queries/query.hbs 22838a4 Diff: https://reviews.apache.org/r/56084/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 56345: Hive2 view UDF tab is blank always

2017-02-06 Thread Pallav Kulshreshtha
b776fd2 contrib/views/hive20/src/main/resources/ui/app/templates/udfs/new.hbs PRE-CREATION Diff: https://reviews.apache.org/r/56345/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Review Request 56384: Hive2: Query Runtime Prediction: Compact Visual Explain Plan

2017-02-07 Thread Pallav Kulshreshtha
://reviews.apache.org/r/56384/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 56443: HiveView2.0: Long table name spills out of table panel

2017-02-08 Thread Pallav Kulshreshtha
://reviews.apache.org/r/56443/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 56443: HiveView2.0: Long table name spills out of table panel

2017-02-08 Thread Pallav Kulshreshtha
://reviews.apache.org/r/56443/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 56430: incorrect imports in hive-next view

2017-02-08 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56430/#review164849 --- Ship it! Ship It! - Pallav Kulshreshtha On Feb. 8, 2017, 6

Review Request 56345: Hive2 view UDF tab is blank always

2017-02-06 Thread Pallav Kulshreshtha
/resources/ui/app/templates/udfs/new.hbs PRE-CREATION Diff: https://reviews.apache.org/r/56345/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 56345: Hive2 view UDF tab is blank always

2017-02-07 Thread Pallav Kulshreshtha
b776fd2 contrib/views/hive20/src/main/resources/ui/app/templates/udfs/new.hbs PRE-CREATION Diff: https://reviews.apache.org/r/56345/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 56383: AMBARI-19889:Workflow Manager Should be able to handle fork with single path.

2017-02-08 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56383/#review164663 --- Ship it! Ship It! - Pallav Kulshreshtha On Feb. 7, 2017, 11

Review Request 56443: HiveView2.0: Long table name spills out of table panel

2017-02-08 Thread Pallav Kulshreshtha
/savedqueries.hbs 0ab6cfd Diff: https://reviews.apache.org/r/56443/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 56443: HiveView2.0: Long table name spills out of table panel

2017-02-08 Thread Pallav Kulshreshtha
/views/hive20/src/main/resources/ui/app/templates/savedqueries.hbs 0ab6cfd Diff: https://reviews.apache.org/r/56443/diff/ Testing --- manually tested. Thanks, Pallav Kulshreshtha

Re: Review Request 55757: AMBARI-19647. Issue while submitting workflow as its not able to register as a project

2017-01-22 Thread Pallav Kulshreshtha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55757/#review162600 --- Ship it! Ship It! - Pallav Kulshreshtha On Jan. 20, 2017

  1   2   3   >