Re: Review Request 64951: Idempotent issue on Ambari Upgrade, renameServiceDeletedColumn failed with column already exists exception

2018-01-05 Thread Vitalyi Brodetskyi
s.apache.org/r/64951/diff/1-2/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Review Request 64951: Idempotent issue on Ambari Upgrade, renameServiceDeletedColumn failed with column already exists exception

2018-01-04 Thread Vitalyi Brodetskyi
n ON clusterconfig (cluster_id, type_name, version); {code} Diffs - ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog252.java e6a7829 Diff: https://reviews.apache.org/r/64951/diff/1/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 64863: Implement resolving dependencies for each service

2018-01-02 Thread Vitalyi Brodetskyi
soon Thanks, Vitalyi Brodetskyi

Review Request 64863: Implement resolving dependencies for each service

2017-12-28 Thread Vitalyi Brodetskyi
://reviews.apache.org/r/64863/diff/1/ Testing --- UT will be added soon Thanks, Vitalyi Brodetskyi

Re: Review Request 64710: UI hangs on step 2 of security wizard

2017-12-28 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64710/#review194573 --- Ship it! Ship It! - Vitalyi Brodetskyi On Гру. 19, 2017, 2

Re: Review Request 64693: Add service group dependencies

2017-12-21 Thread Vitalyi Brodetskyi
/ Testing --- Thanks, Vitalyi Brodetskyi

Review Request 64782: Add service dependencies

2017-12-21 Thread Vitalyi Brodetskyi
, Vitalyi Brodetskyi

Re: Review Request 64709: Stack unit tests fixes

2017-12-19 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64709/#review194140 --- Ship it! Ship It! - Vitalyi Brodetskyi On Гру. 19, 2017, 12

Review Request 64693: Add service group dependencies

2017-12-18 Thread Vitalyi Brodetskyi
Diff: https://reviews.apache.org/r/64693/diff/1/ Testing --- Thanks, Vitalyi Brodetskyi

Re: Review Request 64682: Fix unit test failures caused by old json format

2017-12-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64682/#review194067 --- Ship it! Ship It! - Vitalyi Brodetskyi On Гру. 18, 2017, 1

Re: Review Request 64606: Fix unit tests in feature branch to make them workable

2017-12-14 Thread Vitalyi Brodetskyi
/StageUtilsTest.java 8546a9b Diff: https://reviews.apache.org/r/64606/diff/2/ Changes: https://reviews.apache.org/r/64606/diff/1-2/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Review Request 64606: Fix unit tests in feature branch to make them workable

2017-12-14 Thread Vitalyi Brodetskyi
: https://reviews.apache.org/r/64606/diff/1/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Review Request 64420: Fix unit tests in feature branch to make them workable

2017-12-07 Thread Vitalyi Brodetskyi
/java/org/apache/ambari/server/state/cluster/ClustersTest.java be4a907 ambari-server/src/test/resources/cluster-settings.xml PRE-CREATION Diff: https://reviews.apache.org/r/64420/diff/1/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 64374: Ambari Builds Are Failing With Python Test Errors

2017-12-06 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64374/#review192978 --- Ship it! Ship It! - Vitalyi Brodetskyi On Гру. 6, 2017, 12

Re: Review Request 64066: Fix validators tests.

2017-11-24 Thread Vitalyi Brodetskyi
, Vitalyi Brodetskyi

Review Request 64066: Fix validators tests.

2017-11-24 Thread Vitalyi Brodetskyi
/ambari/server/upgrade/AbstractUpgradeCatalogTest.java d7ad7f2 ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog300Test.java 03219bf Diff: https://reviews.apache.org/r/64066/diff/1/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 64005: Fix build for test classes in feature branch

2017-11-22 Thread Vitalyi Brodetskyi
/apache/ambari/server/view/ViewRegistryTest.java a28d419 Diff: https://reviews.apache.org/r/64005/diff/2/ Changes: https://reviews.apache.org/r/64005/diff/1-2/ Testing --- Just mvn rpm:rpm. Thanks, Vitalyi Brodetskyi

Review Request 64005: Fix build for test classes in feature branch

2017-11-21 Thread Vitalyi Brodetskyi
Diff: https://reviews.apache.org/r/64005/diff/1/ Testing --- Just mvn rpm:rpm. Thanks, Vitalyi Brodetskyi

Re: Review Request 63872: NFSGateway start failed

2017-11-16 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63872/#review191166 --- Ship it! Ship It! - Vitalyi Brodetskyi On Лис. 16, 2017, 10

Review Request 63819: Ambari Infra, AMS, LogSearch are missing from list of services to be installed

2017-11-14 Thread Vitalyi Brodetskyi
://reviews.apache.org/r/63819/diff/1/ Testing --- Thanks, Vitalyi Brodetskyi

Review Request 63742: Add service dependencies

2017-11-10 Thread Vitalyi Brodetskyi
/r/63742/diff/1/ Testing --- Thanks, Vitalyi Brodetskyi

Re: Review Request 63701: Upstart is not able to stop the ambari-agent

2017-11-09 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63701/#review190586 --- Ship it! Ship It! - Vitalyi Brodetskyi On Ноя. 9, 2017, 2

Re: Review Request 63464: Beeline connection hangs for longer time when connection to HS2 with metastore DB down

2017-11-09 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63464/#review190585 --- Ship it! Ship It! - Vitalyi Brodetskyi On Ноя. 1, 2017, 12

Re: Review Request 63653: Add service group dependencies

2017-11-08 Thread Vitalyi Brodetskyi
/diff/1-2/ Testing --- Thanks, Vitalyi Brodetskyi

Review Request 63653: Add service group dependencies

2017-11-07 Thread Vitalyi Brodetskyi
-CREATE.sql dfef1ce ambari-server/src/main/resources/META-INF/persistence.xml 009ecd5 Diff: https://reviews.apache.org/r/63653/diff/1/ Testing --- Thanks, Vitalyi Brodetskyi

Re: Review Request 63444: A bunch of alerts fails due to absence of configs

2017-11-06 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63444/#review190142 --- Ship it! Ship It! - Vitalyi Brodetskyi On Лис. 2, 2017, 1

Re: Review Request 63504: Changing hostname causes ambari-agent start to fail

2017-11-02 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63504/#review189912 --- Ship it! Ship It! - Vitalyi Brodetskyi On Лис. 2, 2017, 11

Review Request 63457: Add service dependencies

2017-10-31 Thread Vitalyi Brodetskyi
-Postgres-CREATE.sql f48bcd6 ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql e7c5c72 ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql f87af4c Diff: https://reviews.apache.org/r/63457/diff/1/ Testing --- Thanks, Vitalyi Brodetskyi

Re: Review Request 63411: PERF stack: UI hangs to 2nd step

2017-10-30 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63411/#review189594 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 30, 2017, 3

Re: Review Request 63412: Alerts UPDATE/DELETE should work deleting and adding a cluster

2017-10-30 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63412/#review189593 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 30, 2017, 3

Re: Review Request 63051: Add service group dependencies

2017-10-25 Thread Vitalyi Brodetskyi
: https://reviews.apache.org/r/63051/diff/7-8/ Testing --- Will fix tests after patch approve Thanks, Vitalyi Brodetskyi

Re: Review Request 63289: Spark history server is stopped (with umask 027 and custom spark log/pid dir)

2017-10-25 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63289/#review189177 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 25, 2017, 8

Re: Review Request 63051: Add service group dependencies

2017-10-19 Thread Vitalyi Brodetskyi
://reviews.apache.org/r/63051/diff/7/ Changes: https://reviews.apache.org/r/63051/diff/6-7/ Testing --- Will fix tests after patch approve Thanks, Vitalyi Brodetskyi

Re: Review Request 63051: Add service group dependencies

2017-10-19 Thread Vitalyi Brodetskyi
://reviews.apache.org/r/63051/diff/6/ Changes: https://reviews.apache.org/r/63051/diff/5-6/ Testing --- Will fix tests after patch approve Thanks, Vitalyi Brodetskyi

Re: Review Request 63051: Add service group dependencies

2017-10-19 Thread Vitalyi Brodetskyi
To reply, visit: https://reviews.apache.org/r/63051/#review188693 --- On Жов. 19, 2017, 2:58 після полудня, Vitalyi Brodetskyi wrote: > > --- > This is an automatically generate

Re: Review Request 63051: Add service group dependencies

2017-10-19 Thread Vitalyi Brodetskyi
://reviews.apache.org/r/63051/diff/5/ Changes: https://reviews.apache.org/r/63051/diff/4-5/ Testing --- Will fix tests after patch approve Thanks, Vitalyi Brodetskyi

Re: Review Request 63051: Add service group dependencies

2017-10-19 Thread Vitalyi Brodetskyi
://reviews.apache.org/r/63051/diff/4/ Changes: https://reviews.apache.org/r/63051/diff/3-4/ Testing --- Will fix tests after patch approve Thanks, Vitalyi Brodetskyi

Re: Review Request 63051: Add service group dependencies

2017-10-18 Thread Vitalyi Brodetskyi
://reviews.apache.org/r/63051/diff/3/ Changes: https://reviews.apache.org/r/63051/diff/2-3/ Testing --- Will fix tests after patch approve Thanks, Vitalyi Brodetskyi

Re: Review Request 63051: Add service group dependencies

2017-10-18 Thread Vitalyi Brodetskyi
> On Жов. 18, 2017, 1:43 після полудня, Vitalyi Brodetskyi wrote: > > Ship It! Wrong review request))) - Vitalyi --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63051/#rev

Re: Review Request 63051: Add service group dependencies

2017-10-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63051/#review188485 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 17, 2017, 8

Re: Review Request 63071: A bunch of services fail to start

2017-10-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63071/#review188484 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 17, 2017, 11

Re: Review Request 63051: Add service group dependencies

2017-10-17 Thread Vitalyi Brodetskyi
Thanks, Vitalyi Brodetskyi

Re: Review Request 63080: Smartsense requires HDFS to be always present while installing

2017-10-17 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63080/#review188332 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 17, 2017, 5

Review Request 63051: Add service group dependencies

2017-10-16 Thread Vitalyi Brodetskyi
-CREATE.sql f48bcd6 Diff: https://reviews.apache.org/r/63051/diff/1/ Testing --- Will fix tests after patch approve Thanks, Vitalyi Brodetskyi

Re: Review Request 62895: HdpCoreMpack: Quick Links should show NN web UI links

2017-10-11 Thread Vitalyi Brodetskyi
- On Жов. 11, 2017, 4:17 після полудня, Vitalyi Brodetskyi wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62895/ >

Re: Review Request 62895: HdpCoreMpack: Quick Links should show NN web UI links

2017-10-11 Thread Vitalyi Brodetskyi
sit: https://reviews.apache.org/r/62895/#review187697 --- On Жов. 11, 2017, 4:17 після полудня, Vitalyi Brodetskyi wrote: > > --- > This is an automatically generated e-mail.

Review Request 62895: HdpCoreMpack: Quick Links should show NN web UI links

2017-10-11 Thread Vitalyi Brodetskyi
/62895/diff/1/ Testing --- will check tests after approve Thanks, Vitalyi Brodetskyi

Re: Review Request 62888: ZKFC start failed due to hadoop-hdfs-zkfc is not supported

2017-10-11 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62888/#review187649 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 11, 2017, 12

Re: Review Request 62808: hadooplzo package installation failed on devdeploys

2017-10-10 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62808/#review187515 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 10, 2017, 11

Re: Review Request 62808: hadooplzo package installation failed on devdeploys

2017-10-06 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62808/#review187256 --- Ship it! Ship It! - Vitalyi Brodetskyi On Жов. 6, 2017, 12

Re: Review Request 62756: Refactor AMS logic in stack advisors to service advisors

2017-10-05 Thread Vitalyi Brodetskyi
/common/test_stack_advisor.py d6b572e Diff: https://reviews.apache.org/r/62756/diff/4/ Changes: https://reviews.apache.org/r/62756/diff/3-4/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 62756: Refactor AMS logic in stack advisors to service advisors

2017-10-04 Thread Vitalyi Brodetskyi
://reviews.apache.org/r/62756/diff/2-3/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 62756: Refactor AMS logic in stack advisors to service advisors

2017-10-04 Thread Vitalyi Brodetskyi
s work with stack advisor. - Vitalyi --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62756/#review187037 --- On Жов. 4, 2017, 8:

Re: Review Request 62756: Refactor AMS logic in stack advisors to service advisors

2017-10-04 Thread Vitalyi Brodetskyi
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62756/#review187036 ------- On Жов. 4, 2017, 8:26 до полудня, Vitalyi Brodetskyi wrote: > > -

Re: Review Request 62756: Refactor AMS logic in stack advisors to service advisors

2017-10-04 Thread Vitalyi Brodetskyi
/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Review Request 62756: Refactor AMS logic in stack advisors to service advisors

2017-10-03 Thread Vitalyi Brodetskyi
-CREATION ambari-server/src/main/resources/common-services/ATLAS/0.7.0.3.0/service_advisor.py a2e31cc Diff: https://reviews.apache.org/r/62756/diff/1/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 51198: Hsi-Hs2: Mondrian tests intermittently fail with "java.io.IOException: Connection reset by peer"

2017-10-02 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51198/#review186837 --- Ship it! Ship It! - Vitalyi Brodetskyi On Сер. 18, 2016, 9

Re: Review Request 61841: Ensure topology_mappings.data has 0644 permissions

2017-10-02 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61841/#review186836 --- Ship it! Ship It! - Vitalyi Brodetskyi On None, Andrew

Re: Review Request 62669: Make dfs.permissions.superusergroup as group property

2017-09-28 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62669/#review186577 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 28, 2017, 4

Re: Review Request 62667: Zeppelin start failed

2017-09-28 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62667/#review186570 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 28, 2017, 3

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Vitalyi Brodetskyi
eviews.apache.org/r/62599/#review186408 --- On Вер. 27, 2017, 9:52 після полудня, Vitalyi Brodetskyi wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Vitalyi Brodetskyi
/stacks/HDP/3.0/hooks/before-ANY/scripts/params.py eb5feae Diff: https://reviews.apache.org/r/62599/diff/2/ Changes: https://reviews.apache.org/r/62599/diff/1-2/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-27 Thread Vitalyi Brodetskyi
oing this. > > Jayush Luniya wrote: > @Madhu > I think we renamed to in the final patch. Can > you confirm? Jayush, you are right. I've tested my patch and it works with - Vitalyi --- This is an automatically

Review Request 62599: Remove user to group hardcoded logic in hooks

2017-09-26 Thread Vitalyi Brodetskyi
Diff: https://reviews.apache.org/r/62599/diff/1/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 62500: File "hdfs.headless.keytab" has incorrect permission

2017-09-22 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62500/#review185966 --- Ship it! Ship It! - Vitalyi Brodetskyi On Сен. 22, 2017, 10

Re: Review Request 62431: Ambari schema upgrade failure as views destination path dir already exists

2017-09-20 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62431/#review185777 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 20, 2017, 8

Re: Review Request 62402: Multiple fixes into branch-3.0-perf

2017-09-19 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62402/#review185662 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 19, 2017, 8

Re: Review Request 62364: Set tez.runtime.shuffle.ssl.enable=false in Ambari for HSI

2017-09-18 Thread Vitalyi Brodetskyi
/diff/1-2/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Review Request 62364: Set tez.runtime.shuffle.ssl.enable=false in Ambari for HSI

2017-09-15 Thread Vitalyi Brodetskyi
/resources/stacks/HDP/2.6/upgrades/nonrolling-upgrade-2.6.xml b49fdbf ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/upgrade-2.6.xml af885e1 Diff: https://reviews.apache.org/r/62364/diff/1/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 61994: Service repoinfo.xml needs to expose component for Ubuntu repos

2017-09-15 Thread Vitalyi Brodetskyi
://reviews.apache.org/r/61994/diff/4-5/ Testing --- test will be fixed after solution will be approved Thanks, Vitalyi Brodetskyi

Re: Review Request 62348: branch-2.6 Builds Fail With Forked VM Error On Tests

2017-09-15 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62348/#review185471 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 15, 2017, 6

Re: Review Request 61994: Service repoinfo.xml needs to expose component for Ubuntu repos

2017-09-14 Thread Vitalyi Brodetskyi
/internal/RepositoryResourceProviderTest.java 3a7b19b Diff: https://reviews.apache.org/r/61994/diff/4/ Changes: https://reviews.apache.org/r/61994/diff/3-4/ Testing --- test will be fixed after solution will be approved Thanks, Vitalyi Brodetskyi

Re: Review Request 62278: Ambari updates database/tables location from intended remote HDFS to local HDFS

2017-09-13 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62278/#review185279 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 13, 2017, 11

Re: Review Request 62249: Do not export YARN_CONF_DIR in 'yarn-env.sh' file

2017-09-12 Thread Vitalyi Brodetskyi
249/diff/2/ Changes: https://reviews.apache.org/r/62249/diff/1-2/ Testing --- TODO Thanks, Vitalyi Brodetskyi

Review Request 62249: Do not export YARN_CONF_DIR in 'yarn-env.sh' file

2017-09-12 Thread Vitalyi Brodetskyi
l aaa72d1 ambari-server/src/main/resources/stacks/HDP/2.6/services/YARN/configuration/yarn-env.xml 121a81d Diff: https://reviews.apache.org/r/62249/diff/1/ Testing --- TODO Thanks, Vitalyi Brodetskyi

Re: Review Request 62215: Ambari should honor permissions specified for dfs.datanode.data.dir.perm when creating datanode dir's

2017-09-11 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62215/#review185069 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 11, 2017, 10

Re: Review Request 61994: Service repoinfo.xml needs to expose component for Ubuntu repos

2017-09-08 Thread Vitalyi Brodetskyi
ring ambari build process. - Vitalyi --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61994/#review184970 --- On Вер. 7, 2017, 8:40

Re: Review Request 61994: Service repoinfo.xml needs to expose component for Ubuntu repos

2017-09-07 Thread Vitalyi Brodetskyi
view184878 ------- On Вер. 7, 2017, 8:40 після полудня, Vitalyi Brodetskyi wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apach

Re: Review Request 61994: Service repoinfo.xml needs to expose component for Ubuntu repos

2017-09-07 Thread Vitalyi Brodetskyi
-2/ Testing --- test will be fixed after solution will be approved Thanks, Vitalyi Brodetskyi

Review Request 62121: Fix AmbariServerTest

2017-09-06 Thread Vitalyi Brodetskyi
/org/apache/ambari/server/checks/DatabaseConsistencyCheckHelper.java f5a7203 Diff: https://reviews.apache.org/r/62121/diff/1/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Vitalyi Brodetskyi
/62072/diff/4/ Changes: https://reviews.apache.org/r/62072/diff/3-4/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Vitalyi Brodetskyi
, probably we don't need them, what do you think? - Vitalyi --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62072/#review184559 -----------

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Vitalyi Brodetskyi
Thanks, Vitalyi Brodetskyi

Re: Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-05 Thread Vitalyi Brodetskyi
Thanks, Vitalyi Brodetskyi

Re: Review Request 62058: Deploys failing with MR service check failure due to missing mapreduce.tar.gz

2017-09-05 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62058/#review184541 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 4, 2017, 9

Review Request 62072: Remove hive.custom-extensions.root from Hive Properties.

2017-09-04 Thread Vitalyi Brodetskyi
c0800ed Diff: https://reviews.apache.org/r/62072/diff/1/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 62026: ambari-server upgrade process still puts the new views jar to default location even if views.dir is customized in ambari.properties

2017-09-01 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62026/#review184348 --- Ship it! Ship It! - Vitalyi Brodetskyi On Вер. 1, 2017, 8

Review Request 61994: Service repoinfo.xml needs to expose component for Ubuntu repos

2017-08-30 Thread Vitalyi Brodetskyi
ambari-server/src/test/java/org/apache/ambari/server/stack/StackModuleTest.java 4698178 Diff: https://reviews.apache.org/r/61994/diff/1/ Testing --- test will be fixed after solution will be approved Thanks, Vitalyi Brodetskyi

Re: Review Request 61969: Database consistency check fails after upgrade to ambari 2.6.0

2017-08-29 Thread Vitalyi Brodetskyi
ff: https://reviews.apache.org/r/61969/diff/2/ Changes: https://reviews.apache.org/r/61969/diff/1-2/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Review Request 61969: Database consistency check fails after upgrade to ambari 2.6.0

2017-08-29 Thread Vitalyi Brodetskyi
esting --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 61941: Fix config paths in libraries folder

2017-08-28 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61941/#review183923 --- Ship it! Ship It! - Vitalyi Brodetskyi On Сер. 28, 2017, 1

Re: Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-25 Thread Vitalyi Brodetskyi
/UpgradeCatalog260Test.java 2e38686 Diff: https://reviews.apache.org/r/61856/diff/5/ Changes: https://reviews.apache.org/r/61856/diff/4-5/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-25 Thread Vitalyi Brodetskyi
/UpgradeCatalog260Test.java 2e38686 Diff: https://reviews.apache.org/r/61856/diff/4/ Changes: https://reviews.apache.org/r/61856/diff/3-4/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-23 Thread Vitalyi Brodetskyi
1856/diff/1/?file=1802401#file1802401line54> > > > > Make sure to run checkstyle on this. done. - Vitalyi --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61856/#review1836

Re: Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-23 Thread Vitalyi Brodetskyi
/UpgradeCatalog260Test.java 44b5d91 Diff: https://reviews.apache.org/r/61856/diff/3/ Changes: https://reviews.apache.org/r/61856/diff/2-3/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Review Request 61856: Auto fix service config mapping issue discovered during ambari-upgrade

2017-08-23 Thread Vitalyi Brodetskyi
clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 61582: Not able to start Yarn services after restoring the configs to initial value

2017-08-21 Thread Vitalyi Brodetskyi
lusterImpl.java 84b411c Diff: https://reviews.apache.org/r/61582/diff/4/ Changes: https://reviews.apache.org/r/61582/diff/3-4/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 61582: Not able to start Yarn services after restoring the configs to initial value

2017-08-21 Thread Vitalyi Brodetskyi
lusterImpl.java 84b411c Diff: https://reviews.apache.org/r/61582/diff/3/ Changes: https://reviews.apache.org/r/61582/diff/2-3/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 61582: Not able to start Yarn services after restoring the configs to initial value

2017-08-21 Thread Vitalyi Brodetskyi
lusterImpl.java 84b411c Diff: https://reviews.apache.org/r/61582/diff/2/ Changes: https://reviews.apache.org/r/61582/diff/1-2/ Testing --- mvn clean test Thanks, Vitalyi Brodetskyi

Re: Review Request 61727: Unable to add new host to the upgraded PPC cluster due to exception:Local OS is not compatible with cluster primary OS family

2017-08-18 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61727/#review183208 --- Ship it! Ship It! - Vitalyi Brodetskyi On Сер. 18, 2017, 7

  1   2   3   4   5   6   >