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

> On May 20, 2016, 4:40 a.m., Ashwin Rajeev wrote: > > contrib/views/hive/src/main/resources/ui/hive-web/app/components/file-upload.js, > > line 30 > > > > > > Remove logging why? I have kept logging in some places

Re: Review Request 47639: Remove unused print from zeppelin service check

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47639/ --- (Updated May 20, 2016, 7:05 a.m.) Review request for Ambari and Pallav Kulshres

Re: Review Request 47639: Remove unused print from zeppelin service check

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47639/ --- (Updated May 20, 2016, 7:06 a.m.) Review request for Ambari and Pallav Kulshres

Re: Review Request 47596: HiveServer interactive - incorrect default memory value

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47596/ --- (Updated máj. 20, 2016, 8:01 de) Review request for Ambari, Laszlo Puskas, Oliv

Review Request 47642: Add HAWQ Standby wizard should recommend HAWQSTANDBY correctly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47642/ --- Review request for Ambari, Alexander Denissov, bhuvnesh chaudhary, and Lav Jain.

Re: Review Request 47596: HiveServer interactive - incorrect default memory value

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47596/#review134122 --- Ship it! Ship It! - Sebastian Toader On May 20, 2016, 10:01

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47604/#review134126 --- Ship it! Ship It! - Nitiraj Rathore On May 20, 2016, 6:57 a.

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47643/ --- Review request for Ambari and Sreenath Somarajapuram. Bugs: AMBARI-16792 ht

Review Request 47644: All created views disappear after upgrade to 2.4.0.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47644/ --- Review request for Ambari, DIPAYAN BHOWMICK, Nitiraj Rathore, Pallav Kulshreshth

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47525/ --- (Updated May 20, 2016, 10:16 a.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 47626: Atlas HA configuration Property Changed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47626/ --- (Updated May 20, 2016, 10:16 a.m.) Review request for Ambari, Erik Bergenholtz

Re: Review Request 47599: Add validation for alert name edit

> On May 19, 2016, 4:08 p.m., Alexandr Antonenko wrote: > > Ship It! Hi Alexander, can you please help push the fix. Thank you. - Sangeeta --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4759

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47525/#review134129 --- Ship it! Ship It! - Ashwin Rajeev On May 20, 2016, 10:16 a.m

Re: Review Request 47626: Atlas HA configuration Property Changed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47626/#review134132 --- Ship it! Ship It! - Erik Bergenholtz On May 20, 2016, 10:16

Re: Review Request 47527: Hive view : Upload Table- marks column as INT since first 6 rows have INT values.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47527/#review134133 --- Ship it! Ship It! - DIPAYAN BHOWMICK On May 20, 2016, 6:58 a

Re: Review Request 47595: Implement in ams collector batch insert operations to ams-hbase

> On Май 20, 2016, 4:21 д.п., Sid Wagle wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/ams-site.xml, > > line 557 > > > > > > Any reason for chosing 3 seconds vs 5 ?

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47524/ --- (Updated May 20, 2016, 11:44 a.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 47595: Implement in ams collector batch insert operations to ams-hbase

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47595/ --- (Updated Май 20, 2016, 11:47 д.п.) Review request for Ambari, Aravindan Vijayan

Re: Review Request 47631: 'Configure Ambari Identity' fails when enabling Kerberos on non-root Ambari server

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47631/#review134135 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 47428: Changes to Phoenix QueryServer Kerberos configuration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47428/#review134136 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 47631: 'Configure Ambari Identity' fails when enabling Kerberos on non-root Ambari server

> On May 20, 2016, 7:57 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/utils/ShellCommandUtil.java, > > lines 207-209 > > > > > > Should this be !WINDOWS? -p works for Mac too Good

Re: Review Request 47631: 'Configure Ambari Identity' fails when enabling Kerberos on non-root Ambari server

> On May 20, 2016, 7:57 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/utils/ShellCommandUtil.java, > > line 213 > > > > > > just runCommand? Its interesting that IntellJ didn't fla

Re: Review Request 47428: Changes to Phoenix QueryServer Kerberos configuration

> On May 20, 2016, 8:10 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java, > > line 1967 > > > > > > Is "/spnego" always correct? Should/can this be fro

Re: Review Request 47631: 'Configure Ambari Identity' fails when enabling Kerberos on non-root Ambari server

> On May 20, 2016, 7:57 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/utils/ShellCommandUtil.java, > > lines 207-209 > > > > > > Should this be !WINDOWS? -p works for Mac too > >

Re: Review Request 47631: 'Configure Ambari Identity' fails when enabling Kerberos on non-root Ambari server

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47631/ --- (Updated May 20, 2016, 10:10 a.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 47631: 'Configure Ambari Identity' fails when enabling Kerberos on non-root Ambari server

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47631/ --- (Updated May 20, 2016, 10:09 a.m.) Review request for Ambari, Jonathan Hurley,

Re: Review Request 47644: All created views disappear after upgrade to 2.4.0.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47644/#review134146 --- Ship it! Ship It! - Ashwin Rajeev On May 20, 2016, 10:15 a.m

Re: Review Request 47428: Changes to Phoenix QueryServer Kerberos configuration

> On May 20, 2016, 12:10 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java, > > lines 1924-1938 > > > > > > This is a bit rough on the eyes. There's not

Re: Review Request 47596: HiveServer interactive - incorrect default memory value

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47596/#review134148 --- Ship it! Ship It! - Oliver Szabo On May 20, 2016, 8:01 a.m.,

Re: Review Request 47596: HiveServer interactive - incorrect default memory value

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47596/#review134149 --- Ship it! Ship It! - Laszlo Puskas On May 20, 2016, 8:01 a.m.

Re: Review Request 47531: Ambari APIs should allow using case insensitive host names

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47531/ --- (Updated Травень 20, 2016, 3:27 після полудня) Review request for Ambari, Dmytr

Re: Review Request 47531: Ambari APIs should allow using case insensitive host names

> On Травень 19, 2016, 2:05 після полудня, Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/api/predicate/QueryParser.java, > > lines 103-105 > > > > > > What about any endpoint that takes h

Re: Review Request 47596: HiveServer interactive - incorrect default memory value

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47596/#review134151 --- Ship it! Ship It! - Sandor Magyari On May 20, 2016, 8:01 a.m

Re: Review Request 47531: Ambari APIs should allow using case insensitive host names

> On Травень 18, 2016, 5:44 після полудня, Alejandro Fernandez wrote: > > Which use cases were tested? E.g., > > > > 1. Add a host with name foo > > 2. Attempt to readd a host with name FOO (should not allow this). > > 3. Query the host components of a host with name Foo > > 4. Delete a host wit

Review Request 47651: Required empty config is present after Upgrade from 2.1.1 to 2.4.0.0 [HDFS: keyserver_port]

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47651/ --- Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Vitalyi Brod

Re: Review Request 47651: Required empty config is present after Upgrade from 2.1.1 to 2.4.0.0 [HDFS: keyserver_port]

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47651/#review134162 --- Ship it! Ship It! - Vitalyi Brodetskyi On Травень 20, 2016,

Re: Review Request 47531: Ambari APIs should allow using case insensitive host names

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47531/#review134164 --- Ship it! Ship It! - Dmytro Sen On Май 20, 2016, 3:27 п.п., V

Re: Review Request 47428: Changes to Phoenix QueryServer Kerberos configuration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47428/#review134165 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 47531: Ambari APIs should allow using case insensitive host names

> On May 18, 2016, 5:44 p.m., Alejandro Fernandez wrote: > > Which use cases were tested? E.g., > > > > 1. Add a host with name foo > > 2. Attempt to readd a host with name FOO (should not allow this). > > 3. Query the host components of a host with name Foo > > 4. Delete a host with name fOO (s

Re: Review Request 47531: Ambari APIs should allow using case insensitive host names(for SmartSense)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47531/ --- (Updated Травень 20, 2016, 5:56 після полудня) Review request for Ambari, Dmytr

Re: Review Request 47531: Ambari APIs should allow using case insensitive host names(for SmartSense)

> On Травень 18, 2016, 5:44 після полудня, Alejandro Fernandez wrote: > > Which use cases were tested? E.g., > > > > 1. Add a host with name foo > > 2. Attempt to readd a host with name FOO (should not allow this). > > 3. Query the host components of a host with name Foo > > 4. Delete a host wit

Review Request 47653: clean up import * for OOZIE service scripts in common-services

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47653/ --- Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Matt. Bugs:

Re: Review Request 47653: clean up import * for OOZIE service scripts in common-services

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47653/#review134175 --- Ship it! Ship It! - Alejandro Fernandez On May 20, 2016, 6:1

Re: Review Request 47428: Changes to Phoenix QueryServer Kerberos configuration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47428/ --- (Updated May 20, 2016, 6:18 p.m.) Review request for Ambari, Jonathan Hurley, N

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47540/ --- (Updated May 20, 2016, 6:21 p.m.) Review request for Ambari, DIPAYAN BHOWMICK,

Re: Review Request 47621: AMBARI-16784. Remove 'org.apache.atlas.hive.hook.HiveHook' from Hive2/hive-site.xml config hive.exec.post.hooks as Atlas is not supported by Hive2.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47621/ --- (Updated May 20, 2016, 6:23 p.m.) Review request for Ambari, Alejandro Fernande

Re: Review Request 47621: AMBARI-16784. Remove 'org.apache.atlas.hive.hook.HiveHook' from Hive2/hive-site.xml config hive.exec.post.hooks as Atlas is not supported by Hive2.

> On May 20, 2016, 2:55 a.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_interactive.py, > > line 97 > > > > > > convert it to a function and remove

Re: Review Request 47332: Upgrade pack changes to work with Storm 1.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47332/#review134178 --- Fix it, then Ship it! Thanks for making this change. Ship It on

Re: Review Request 47621: AMBARI-16784. Remove 'org.apache.atlas.hive.hook.HiveHook' from Hive2/hive-site.xml config hive.exec.post.hooks as Atlas is not supported by Hive2.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47621/#review134181 --- Ship it! Ship It! - Alejandro Fernandez On May 20, 2016, 6:2

Re: Review Request 47621: AMBARI-16784. Remove 'org.apache.atlas.hive.hook.HiveHook' from Hive2/hive-site.xml config hive.exec.post.hooks as Atlas is not supported by Hive2.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47621/#review134185 --- Ship it! Ship It! - Sumit Mohanty On May 20, 2016, 6:23 p.m.

Review Request 47657: Hawq password to use !h flag to ensure that its only hidden and no special processing is done for it.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47657/ --- Review request for Ambari, Alexander Denissov, Lav Jain, and Matt. Bugs: AMBARI

Re: Review Request 47657: Hawq password to use !h flag to ensure that its only hidden and no special processing is done for it.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47657/#review134186 --- Ship it! Ship It! - Matt On May 20, 2016, 11:59 a.m., bhuvne

Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47656/ --- Review request for Ambari, Alexander Denissov, Alejandro Fernandez, bhuvnesh cha

Re: Review Request 47627: hawq_standby_address_host property should be removed from configuration if HAWQSTANDBY component does not exist in BP

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47627/#review134187 --- Ship it! Ship It! - Matt On May 19, 2016, 7:09 p.m., bhuvnes

Re: Review Request 47656: AMBARI-12885 - Dynamic stack extensions - install and upgrade support for custom services

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47656/ --- (Updated May 20, 2016, 7:03 p.m.) Review request for Ambari, Alexander Denissov

Re: Review Request 47642: Add HAWQ Standby wizard should recommend HAWQSTANDBY correctly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47642/#review134188 --- Ship it! Ship It! - bhuvnesh chaudhary On May 20, 2016, 8:13

Re: Review Request 47332: Upgrade pack changes to work with Storm 1.0

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47332/ --- (Updated May 20, 2016, 7:17 p.m.) Review request for Ambari, Alejandro Fernande

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47643/#review134190 --- contrib/views/capacity-scheduler/src/main/resources/ui/app/templa

Re: Review Request 47631: 'Configure Ambari Identity' fails when enabling Kerberos on non-root Ambari server

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47631/#review134194 --- Ship it! Ship It! - Nate Cole On May 20, 2016, 10:10 a.m., R

Review Request 47669: import missing check_process_status

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47669/ --- Review request for Ambari, Alejandro Fernandez and Jayush Luniya. Bugs: AMBARI-

Re: Review Request 47657: Hawq password to use !h flag to ensure that its only hidden and no special processing is done for it.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47657/#review134207 --- Ship it! Ship It! - Lav Jain On May 20, 2016, 6:59 p.m., bhu

Re: Review Request 47627: hawq_standby_address_host property should be removed from configuration if HAWQSTANDBY component does not exist in BP

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47627/#review134208 --- ambari-server/src/test/java/org/apache/ambari/server/controller/i

Re: Review Request 47627: hawq_standby_address_host property should be removed from configuration if HAWQSTANDBY component does not exist in BP

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47627/ --- (Updated May 20, 2016, 10:11 p.m.) Review request for Ambari, Alexander Denisso

Review Request 47673: Wrong memory conf in spark-env.xml

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47673/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-16715 https://iss

Re: Review Request 47627: hawq_standby_address_host property should be removed from configuration if HAWQSTANDBY component does not exist in BP

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47627/#review134209 --- Ship it! Ship It! - Lav Jain On May 20, 2016, 10:11 p.m., bh

Review Request 47676: Spark History Server heap size is not exposed (History Server crashed with OOM)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47676/ --- Review request for Ambari and Sumit Mohanty. Bugs: AMBARI-16757 https://iss

Re: Review Request 47676: Spark History Server heap size is not exposed (History Server crashed with OOM)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47676/ --- (Updated May 20, 2016, 10:24 p.m.) Review request for Ambari and Sumit Mohanty.

Review Request 47677: AMBARI-16804 - Refactor Move Master Wizard

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47677/ --- Review request for Ambari, Jaimin Jetly, Srimanth Gunturi, and Yusaku Sako. Bug

Re: Review Request 47673: Wrong memory conf in spark-env.xml

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47673/#review134213 --- Ship it! Ship It! - Sumit Mohanty On May 20, 2016, 10:15 p.m

Re: Review Request 47642: Add HAWQ Standby wizard should recommend HAWQSTANDBY correctly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47642/#review134212 --- ambari-server/src/main/resources/common-services/HAWQ/2.0.0/servi

Review Request 47679: PXF operations considers the agent status

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47679/ --- Review request for Ambari, Alexandr Antonenko and Yusaku Sako. Bugs: AMBARI-166

Review Request 47680: AMBARI-16805. Fixing the failing Hive Service check for 'Hive Server Interactive' component

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47680/ --- Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Bugs: AMBARI-

Re: Review Request 47642: Add HAWQ Standby wizard should recommend HAWQSTANDBY correctly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47642/ --- (Updated May 20, 2016, 5:07 p.m.) Review request for Ambari, Alexander Denissov

Re: Review Request 47680: AMBARI-16805. Fixing the failing Hive Service check for 'Hive Server Interactive' component

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47680/#review134223 --- Ship it! Ship It! - Sumit Mohanty On May 21, 2016, 12:04 a.m

Re: Review Request 47679: PXF operations considers the agent status

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47679/#review134226 --- Ship it! Ship It! - bhuvnesh chaudhary On May 20, 2016, 11:2

Review Request 47688: AMBARI-16806: AMS Collector is not automatically being configured for auto-start during blueprint deployment

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47688/ --- Review request for Ambari, Jonathan Hurley, Sumit Mohanty, and Sid Wagle. Bugs: