Re: Review Request 44457: AMBARI-15310: Host Service Summary Page does not display long service names well

2016-03-07 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44457/#review122361 --- Ship it! Ship It! - Alexandr Antonenko On March 7, 2016, 4

Re: Review Request 44253: Add a "Add" button to Repo management UI.

2016-03-04 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44253/#review122083 --- Ship it! Ship It! - Alexandr Antonenko On March 4, 2016, 12

Re: Review Request 44329: Ambari server does not honor the gzip compression correctly

2016-03-03 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44329/#review121842 --- Ship it! Ship It! - Alexandr Antonenko On March 3, 2016, 11

Re: Review Request 44074: Authorization for Auto start services

2016-02-26 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44074/#review120872 --- Ship it! Ship It! - Alexandr Antonenko On Feb. 26, 2016, 4

Re: Review Request 43812: Use ember functions mapProperty and pushObjects in AMBARI-14987

2016-02-25 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43812/#review120691 --- Ship it! Ship It! - Alexandr Antonenko On Feb. 23, 2016, 8

Re: Review Request 43430: assign_master_component.js does not show recommendations while adding master component for a service which is already installed

2016-02-23 Thread Alexandr Antonenko
> On Feb. 19, 2016, 6:57 p.m., Alexandr Antonenko wrote: > > ambari-web/app/mixins/wizard/assign_master_components.js, line 776 > > <https://reviews.apache.org/r/43430/diff/3/?file=1252149#file1252149line776> > > > > No need in such constructio

Re: Review Request 43430: assign_master_component.js does not show recommendations while adding master component for a service which is already installed

2016-02-19 Thread Alexandr Antonenko
mapProperty ambari-web/app/mixins/wizard/assign_master_components.js (line 776) <https://reviews.apache.org/r/43430/#comment181326> No need in such construction, there is native ember function pushObjects. - Alexandr Antonenko On Feb. 16, 2016, 10:1

Re: Review Request 43386: AMBARI-14983 Auto-start services: Show list of Services/Component with status indicator

2016-02-09 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43386/#review118522 --- Ship it! Ship It! - Alexandr Antonenko On Feb. 10, 2016, 12

Re: Review Request 43386: AMBARI-14983 Auto-start services: Show list of Services/Component with status indicator

2016-02-09 Thread Alexandr Antonenko
tps://reviews.apache.org/r/43386/#comment179769> is there a way an easy to not use "important" here? - Alexandr Antonenko On Feb. 9, 2016, 11:17 p.m., Zhe (Joe) Wang wrote: > > --- > This is an automatically generate

Re: Review Request 43154: AMBARI-14766: Selecting ALL groups does not work under Manage Alert Notifications

2016-02-08 Thread Alexandr Antonenko
/manage_alert_notifications_controller.js (line 705) <https://reviews.apache.org/r/43154/#comment179434> how can apiObject get null value ? why are you checking it against null ? - Alexandr Antonenko On Feb. 4, 2016, 5:49 p.m., Keta Patel

Re: Review Request 43268: Dependencies popup is displayed even when all dependent services are selected

2016-02-08 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43268/#review118220 --- Ship it! Ship It! - Alexandr Antonenko On Feb. 5, 2016, 10

Re: Review Request 43055: AMBARI-14403: Improve Quicklink logic to remove any unnecessary legacy code

2016-02-08 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43055/#review118211 --- Ship it! Ship It! - Alexandr Antonenko On Feb. 5, 2016, 2

Re: Review Request 43189: AMBARI-14871: Better messages when Ambari web UI can't resolve hosts to build quicklinks

2016-02-05 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43189/#review118030 --- Ship it! Ship It! - Alexandr Antonenko On Feb. 5, 2016, 4

Re: Review Request 43189: AMBARI-14871: Better messages when Ambari web UI can't resolve hosts to build quicklinks

2016-02-05 Thread Alexandr Antonenko
) <https://reviews.apache.org/r/43189/#comment179308> Why are you making a compare with null ? I do not see any posible way oozieHostsArray can get null value - Alexandr Antonenko On Feb. 5, 2016, 2:39 p.m., Di Li wrote: > > --

Re: Review Request 43000: Longer cluster names extend beyond the frame in the Ambari Admin view.

2016-02-02 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43000/#review117427 --- Ship it! - Alexandr Antonenko On Feb. 2, 2016, 5:36 p.m

Re: Review Request 43000: Longer cluster names extend beyond the frame in the Ambari Admin view.

2016-02-02 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43000/#review117424 --- can you please updare the patch here - Alexandr Antonenko On

Re: Review Request 42976: Rename AdminSetting API to Setting

2016-02-01 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42976/#review117311 --- Ship it! Ship It! - Alexandr Antonenko On Фев. 1, 2016, 9

Re: Review Request 42367: AMBARI-14574: multiple clicks on "Next" button causes skipping of steps while installing a cluster

2016-01-25 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42367/#review116094 --- Ship it! Ship It! - Alexandr Antonenko On Jan. 22, 2016, 7

Re: Review Request 42585: Service deploy wizard is not always restored to the correct state If browser session is lost while deploy is running

2016-01-21 Thread Alexandr Antonenko
> On Jan. 21, 2016, 5:44 p.m., Alexandr Antonenko wrote: > > Ship It! > > Sangeeta Ravindran wrote: > Thank you Alexandr. Can you please help push the fix? yes, ofc - Alexandr --- This is an automatically ge

Re: Review Request 42585: Service deploy wizard is not always restored to the correct state If browser session is lost while deploy is running

2016-01-21 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42585/#review115652 --- Ship it! Ship It! - Alexandr Antonenko On Jan. 21, 2016, 5:30

Re: Review Request 42585: Service deploy wizard is not always restored to the correct state If browser session is lost while deploy is running

2016-01-21 Thread Alexandr Antonenko
> On Jan. 21, 2016, 3:13 p.m., Alexandr Antonenko wrote: > > ambari-web/app/routes/add_service_routes.js, line 85 > > <https://reviews.apache.org/r/42585/diff/1/?file=1204158#file1204158line85> > > > > Why we need to remove that line ? > > Sang

Re: Review Request 42585: Service deploy wizard is not always restored to the correct state If browser session is lost while deploy is running

2016-01-21 Thread Alexandr Antonenko
tps://reviews.apache.org/r/42585/#comment176621> Why we need to remove that line ? - Alexandr Antonenko On Jan. 21, 2016, 5:30 a.m., Sangeeta Ravindran wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 42367: AMBARI-14574: multiple clicks on "Next" button causes skipping of steps while installing a cluster

2016-01-21 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42367/#review115633 --- Ship it! Ship It! - Alexandr Antonenko On Jan. 21, 2016, 2:45

Re: Review Request 42367: AMBARI-14574: multiple clicks on "Next" button causes skipping of steps while installing a cluster

2016-01-21 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42367/#review115634 --- Ship it! Ship It! - Alexandr Antonenko On Jan. 21, 2016, 2:45

Re: Review Request 42367: AMBARI-14574: multiple clicks on "Next" button causes skipping of steps while installing a cluster

2016-01-18 Thread Alexandr Antonenko
/browse/AMBARI-14574?focusedCommentId=15105485&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15105485 - Alexandr Antonenko On Jan. 18, 2016, 4:57 p.m., Keta Patel wrote: > > --- > This is an

Re: Review Request 42006: DB model for admin setting

2016-01-15 Thread Alexandr Antonenko
> On Jan. 15, 2016, 8:51 p.m., Alexandr Antonenko wrote: > > Ship It! Seems good to me, but leaving last word for BE guru - Alexandr --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.or

Re: Review Request 42006: DB model for admin setting

2016-01-15 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42006/#review114760 --- Ship it! Ship It! - Alexandr Antonenko On Jan. 15, 2016, 1:14

Re: Review Request 41577: AMBARI-11670: Width of services menu container is fixed and causes part of longer service names to be rendered outside the frame

2015-12-29 Thread Alexandr Antonenko
ls.com/cssref/css3_pr_word-wrap.asp I was talking about that property :) Please update patch to keep reference. And I will do the commit. Thanks. - Alexandr Antonenko On Dec. 29, 2015, 1:14 a.m., Keta Patel wrote: > > ---

Re: Review Request 41577: AMBARI-11670: Width of services menu container is fixed and causes part of longer service names to be rendered outside the frame

2015-12-29 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41577/#review112210 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 29, 2015, 1:14

Re: Review Request 41668: View display name overlaps with version if name is long and does not contain spaces

2015-12-28 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41668/#review112033 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 23, 2015, 6:27

Re: Review Request 41621: In the Move Master Wizard, manual commands text scroll outside the frame when the message is long

2015-12-28 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41621/#review112032 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 22, 2015, 1:08

Re: Review Request 41666: Typo in error message displayed when connection fails in Hive view

2015-12-22 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41666/#review111687 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 22, 2015, 11:17

Re: Review Request 41577: AMBARI-11670: Width of services menu container is fixed and causes part of longer service names to be rendered outside the frame

2015-12-21 Thread Alexandr Antonenko
in ambari-web/app/views/main/service/menu.js by changing alertsCount : alertsCount: function () { return this.get('content.alertsCount') >99 ? "99+" : this.get('content.alertsCount') ; }.property('content.alertsCou

Re: Review Request 41551: Add Service wizard: Ambari web should select slaves as per the recommendation given by stack advisor

2015-12-18 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41551/#review79 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 18, 2015, 1:02

Re: Review Request 41232: Add service wizard hangs up on Customize Services step on secure cluster after RU

2015-12-10 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41232/#review109893 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 11, 2015, 12:05

Re: Review Request 40890: Unit tests for the PXF service under common-services

2015-12-02 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40890/#review108761 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 3, 2015, 12:53

Re: Review Request 40682: PXF should get secured when security is enabled on cluster via kerberos wizard on ambari

2015-11-30 Thread Alexandr Antonenko
> On Nov. 25, 2015, 1:11 a.m., jun aoki wrote: > > ambari-server/src/main/resources/common-services/PXF/3.0.0/package/scripts/pxf.py, > > line 106 > > > > > > I noticed ConfigDictionary is used before it is passed t

Re: Review Request 40311: AMBARI-11964: Allow services to hide "custom" configurations

2015-11-17 Thread Alexandr Antonenko
> On Nov. 17, 2015, 3:07 p.m., Alexandr Antonenko wrote: > > Ship It! wow, sorry, did not saw that reviewer was changed to Jaimin J. Please ignore my +1. - Alexandr --- This is an automatically generated e-mail. To reply, vis

Re: Review Request 40311: AMBARI-11964: Allow services to hide "custom" configurations

2015-11-17 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40311/#review106866 --- Ship it! Ship It! - Alexandr Antonenko On Nov. 16, 2015, 5:16

Re: Review Request 39232: Incorrect accounts names on Admin->Service Account page

2015-10-12 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39232/#review102252 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 12, 2015, 2:40

Re: Review Request 38927: AMBARI-13211: Host checks does not report all errors in the very first check after Ambari Agent gets installed/registered

2015-10-05 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38927/#review101472 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 5, 2015, 12:40

Re: Review Request 38561: AMBARI-13154: Label of required fields in Details section of Create View Instance UI should include an asterisk

2015-09-25 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38561/#review100569 --- Ship it! Ship It! - Alexandr Antonenko On Sept. 21, 2015, 5:18

Re: Review Request 38525: Master heap setting in distributed mode should be limited

2015-09-19 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38525/#review99682 --- Ship it! Ship It! - Alexandr Antonenko On Sept. 19, 2015, 3:01

Re: Review Request 36991: Ambari upgrade to 2.1 fails (thinks hbase is installed)

2015-07-31 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36991/#review93732 --- Ship it! Ship It! - Alexandr Antonenko On July 31, 2015, 4:14

Review Request 35137: Atlas service UI enhancements

2015-06-05 Thread Alexandr Antonenko
10bc923 ambari-web/app/views/common/quick_view_link_view.js 09176ae Diff: https://reviews.apache.org/r/35137/diff/ Testing --- 6236 tests complete Thanks, Alexandr Antonenko

Re: Review Request 34847: HostCleanup script does not address all issues

2015-05-30 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34847/#review85871 --- Ship it! Ship It! - Alexandr Antonenko On May 30, 2015, 5:22

Re: Review Request 34600: [WinTP2] HDPWIN 2.2 Hive configuration: The MSSQL Server Database Host field is unavailable

2015-05-22 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34600/#review84931 --- Ship it! Ship It! - Alexandr Antonenko On May 22, 2015, 4:46

Re: Review Request 34434: Replace spinner with licensed spinner.gif

2015-05-19 Thread Alexandr Antonenko
/hive-web/tests/img/spinner.gif 39fcb67 Diff: https://reviews.apache.org/r/34434/diff/ Testing --- Manually tested Thanks, Alexandr Antonenko

Review Request 34434: Replace spinner with licensed spinner.gif

2015-05-19 Thread Alexandr Antonenko
Diff: https://reviews.apache.org/r/34434/diff/ Testing --- Manually tested Thanks, Alexandr Antonenko

Re: Review Request 33978: Kerberos Wizard: Downloaded CSV file contains only default data

2015-05-08 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33978/#review83043 --- Ship it! Ship It! - Alexandr Antonenko On May 8, 2015, 1:59 p.m

Re: Review Request 33907: Kerberos: "Attention: Some configurations need your attention" msg in Enable Kerb Wizard hides fields

2015-05-07 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33907/#review82882 --- Ship it! Ship It! - Alexandr Antonenko On May 6, 2015, 7:50 p.m

Re: Review Request 33912: Kerberos: no need for Manage Identities checkbox

2015-05-07 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33912/#review82881 --- Ship it! Ship It! - Alexandr Antonenko On May 6, 2015, 8:30 p.m

Re: Review Request 32308: Creating "false" local user

2015-03-20 Thread Alexandr Antonenko
eview77241 --- On March 20, 2015, 3:42 p.m., Alexandr Antonenko wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.o

Review Request 32308: Creating "false" local user

2015-03-20 Thread Alexandr Antonenko
f014f8d ambari-web/app/utils/config.js ed34232 Diff: https://reviews.apache.org/r/32308/diff/ Testing --- Thanks, Alexandr Antonenko

Review Request 32243: Hive View: Maintain one session per tab

2015-03-19 Thread Alexandr Antonenko
/resources/jobs/AggregatorTest.java 08e8b81 contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/JobServiceTest.java 1c3444e Diff: https://reviews.apache.org/r/32243/diff/ Testing --- Thanks, Alexandr Antonenko

Re: Review Request 31729: UI doesn't send PUT request if last config is removed from "slider-client"

2015-03-04 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31729/#review75185 --- Ship it! Ship It! - Alexandr Antonenko On March 4, 2015, 4:55

Re: Review Request 31365: Start and Test services at enable security shows 100% even though its not completed

2015-02-24 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31365/#review73841 --- Ship it! Ship It! - Alexandr Antonenko On Feb. 24, 2015, 5:59

Re: Review Request 31050: Required Properties in Configure Kerberos Step

2015-02-14 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31050/#review72493 --- Ship it! Ship It! - Alexandr Antonenko On Feb. 14, 2015, 1:47

Review Request 29318: View: Files, cleanup and rendering optimization

2014-12-22 Thread Alexandr Antonenko
/files/src/main/resources/ui/vendor/js/ember-cloaking.js PRE-CREATION pom.xml 5d55074 Diff: https://reviews.apache.org/r/29318/diff/ Testing --- Thanks, Alexandr Antonenko

Re: Review Request 29049: Minor UI changes for enabling AMS

2014-12-15 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29049/#review65081 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 15, 2014, 12:05

Review Request 28856: View: CapScheduler enhancements, cleanup

2014-12-09 Thread Alexandr Antonenko
s 0a52ffa contrib/views/capacity-scheduler/src/main/resources/ui/app/views/queues.js PRE-CREATION contrib/views/capacity-scheduler/src/main/resources/ui/config.coffee 7bd9fa5 Diff: https://reviews.apache.org/r/28856/diff/ Testing --- Thanks, Alexandr Antonenko

Re: Review Request 28680: Views: Files, UI and trash

2014-12-04 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28680/#review63778 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 3, 2014, 11:26

Re: Review Request 28688: AMBARI-7121: Views: CapScheduler service endpoint for operator

2014-12-04 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28688/#review63831 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 4, 2014, 2:26

Re: Review Request 28550: Pig View: readme + blueprint, minor UI

2014-12-01 Thread Alexandr Antonenko
))) - Alexandr Antonenko On Dec. 1, 2014, 1:07 p.m., Jeff Sposetti wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 28550: Pig View: readme + blueprint, minor UI

2014-12-01 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28550/#review63362 --- Ship it! Ship It! - Alexandr Antonenko On Dec. 1, 2014, 1:07

Re: Review Request 28550: Pig View: readme + blueprint, minor UI

2014-12-01 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28550/#review63361 --- Yes of course you are totally right Jeff - Alexandr Antonenko On

Re: Review Request 28550: Pig View: readme + blueprint, minor UI

2014-12-01 Thread Alexandr Antonenko
/pig/blueprint.json, contrib/views/pig/clustertemplate.json - Alexandr Antonenko On Dec. 1, 2014, 1:07 p.m., Jeff Sposetti wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Review Request 28473: Views: Files download of ZIP fails on folder with no permissions

2014-11-26 Thread Alexandr Antonenko
1 contrib/views/files/src/test/java/org/apache/ambari/view/filebrowser/FilebrowserTest.java 6eb10e3 Diff: https://reviews.apache.org/r/28473/diff/ Testing --- Thanks, Alexandr Antonenko

Re: Review Request 27984: Jobs View UI link directs to last Tez Job seen instead of the one clicked

2014-11-13 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27984/#review61233 --- Ship it! Ship It! - Alexandr Antonenko On Nov. 13, 2014, 12:11

Re: Review Request 27793: Hosts page shows only 25 hosts after logging out and logging back in if page size is set to 50 or 100

2014-11-09 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27793/#review60519 --- Ship it! Ship It! - Alexandr Antonenko On Nov. 9, 2014, 9:33

Re: Review Request 27791: Admin View: Issues with user/group permission edit control in Safari + IE10

2014-11-09 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27791/#review60512 --- Ship it! Ship It! - Alexandr Antonenko On Nov. 9, 2014, 4:50

Re: Review Request 27733: Install Wizard > Customize Services > Config Groups

2014-11-07 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27733/#review60344 --- Ship it! Ship It! - Alexandr Antonenko On Nov. 7, 2014, 5:25

Re: Review Request 27491: Properties which added from Add Service Wizard NOT exist after deployed

2014-11-02 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27491/#review59532 --- Ship it! Ship It! - Alexandr Antonenko On Nov. 2, 2014, 8:11

Re: Review Request 27489: Fix Tez configuration section on Ambari UI

2014-11-02 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27489/#review59530 --- Ship it! Ship It! - Alexandr Antonenko On Ноя. 2, 2014, 6:47

Re: Review Request 27488: Configs: not prompted when changes are not saved when clicking on scv

2014-11-02 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27488/#review59524 --- Ship it! Ship It! - Alexandr Antonenko On Nov. 2, 2014, 4:37

Re: Review Request 27487: Ambari not showing HDFS Block status metrics of active namenode after a failover

2014-11-02 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27487/#review59523 --- Ship it! Ship It! - Alexandr Antonenko On Ноя. 2, 2014, 1:50

Re: Review Request 27486: Hide "New MySQL Database" option for Hive in Install Wizard / Add Service Wizard if OS family is redhat5 or sles11

2014-11-02 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27486/#review59520 --- Ship it! Ship It! - Alexandr Antonenko On Ноя. 2, 2014, 12:32

Re: Review Request 27138: Manage Ambari and Dashboard should NOT open in named tabs

2014-11-01 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27138/#review59501 --- Ship it! Ship It! - Alexandr Antonenko On Nov. 1, 2014, 8:36

Re: Review Request 27463: Alerts UI: Summary Page. Create Actions menu

2014-11-01 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27463/#review59495 --- Ship it! Ship It! - Alexandr Antonenko On Nov. 1, 2014, 12:51

Re: Review Request 27139: [Perf400] Hive and other tabs rendering too slow during Customize service wizard

2014-10-24 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27139/#review58275 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 24, 2014, 2:37

Re: Review Request 27138: Manage Ambari and Dashboard should NOT open in named tabs

2014-10-24 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27138/#review58273 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 24, 2014, 2:31

Re: Review Request 27079: Ambari Kafka charts on service page

2014-10-23 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27079/#review57987 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 23, 2014, 1:51

Re: Review Request 27076: Unable to go to Manage Ambari from Installer step9

2014-10-23 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27076/#review57979 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 23, 2014, 1:11

Re: Review Request 27069: Admin View: description is being variable replaced

2014-10-23 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27069/#review57972 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 23, 2014, 9:27

Re: Review Request 27042: Admin View: Provide identical cluster name validation in Manage

2014-10-22 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27042/#review57804 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 22, 2014, 3:29

Re: Review Request 27031: Slider view is not displayed in IE10

2014-10-22 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27031/#review57778 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 22, 2014, 12:48

Re: Review Request 26979: Admin View: attempting to create instance using same instance name fails

2014-10-21 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26979/#review57592 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 21, 2014, 3:46

Re: Review Request 26978: Slider view should show validation messages on main Slider Apps page

2014-10-21 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26978/#review57593 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 21, 2014, 3:46

Re: Review Request 26980: Add support for YARN registry - required for Slider

2014-10-21 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26980/#review57591 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 21, 2014, 4:01

Re: Review Request 26972: Add Service should not be setting HDFS configs when HDFS is not present

2014-10-21 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26972/#review57579 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 21, 2014, 2:44

Re: Review Request 26883: Slider View: Show slider-view user beside the Create App page

2014-10-17 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26883/#review57184 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 17, 2014, 6:23

Re: Review Request 26875: Slider View: metrics and metric ui links should be at the and of the list

2014-10-17 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26875/#review57153 --- Ship it! Ship It! - Alexandr Antonenko On Oct. 17, 2014, 1:50

Re: Review Request 23854: Usability: HDFS Total Space Utilization chart data needs more precision

2014-07-24 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23854/#review48646 --- Ship it! Ship It! - Alexandr Antonenko On July 23, 2014, 1:29

Re: Review Request 22684: Behavior change: host filtering no longer handles startsWith matches

2014-06-17 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22684/#review45940 --- Ship it! Ship It! - Alexandr Antonenko On June 17, 2014, 2:46

Re: Review Request 22265: UI: treat structured_out as JSON, not stringified JSON

2014-06-05 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22265/#review44823 --- Ship it! Ship It! - Alexandr Antonenko On June 5, 2014, 4:44

Re: Review Request 22261: Convert UI code of Add Host and Add Service Wizards to to lazily load host and host component info

2014-06-05 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22261/#review44809 --- Ship it! Ship It! - Alexandr Antonenko On June 5, 2014, 11:20

Re: Review Request 22259: Convert UI code of configs and config groups to lazy loading hosts and hosts component info

2014-06-05 Thread Alexandr Antonenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22259/#review44808 --- Ship it! Ship It! - Alexandr Antonenko On June 5, 2014, 10:12

Review Request 21860: UI not showing Failed task on large cluster (filter + paging issue)

2014-05-23 Thread Alexandr Antonenko
/utils/host_progress_popup.js 2176a4a Diff: https://reviews.apache.org/r/21860/diff/ Testing --- Thanks, Alexandr Antonenko

Review Request 21856: Add unit tests for views/*.js files

2014-05-23 Thread Alexandr Antonenko
ambari-web/test/views/experimental_test.js PRE-CREATION ambari-web/test/views/installer_test.js PRE-CREATION ambari-web/test/views/login_test.js PRE-CREATION Diff: https://reviews.apache.org/r/21856/diff/ Testing --- Thanks, Alexandr Antonenko

Review Request 20715: UI does not redirect on 'Dashboard' page after login

2014-04-25 Thread Alexandr Antonenko
b1 ambari-web/app/routes/installer.js 3991c57 ambari-web/app/routes/main.js e305cab Diff: https://reviews.apache.org/r/20715/diff/ Testing --- Thanks, Alexandr Antonenko

  1   2   >