---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56980/#review167261
---
Ship it!
Ship It!
- Sebastian Toader
On March 1, 2017, 12
/package/scripts/setup_ranger_xml.py
Lines 761 (patched)
<https://reviews.apache.org/r/57155/#comment239398>
Why wee need this if there is already a ```@retry``` annotation of the
```check_znode``` method?
- Sebastian Toader
On Feb. 28, 2017, 7:47 p.m., Oliver Szabo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57145/#review167071
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 28, 2017, 2:27
he whole syprep from config in a central
way as well. The flags under cluster-env can be modified only either changing
the Blueprint or changing the existing cluster-env config. The flags under
cluster-env are rather for fine tuning the 'sysprep' once enabled.
- Sebastian Toader
, Failures: 0, Errors: 0, Skipped: 39
Thanks,
Sebastian Toader
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57039/#review166805
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 24, 2017, 10
On Feb. 24, 2017, 5:27 p.m., Sebastian Toader wrote:
> > Should any unit tests be updated?
Existing unit tests passed fine with the change. Is there already a mechanism
in Ambari that can be used to test if certain records are persisted within a
transaction or not?
- Seb
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57040/#review166721
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 24, 2017, 5:18
/TopologyManager.java
a26624e
Diff: https://reviews.apache.org/r/56987/diff/
Testing
---
Performed manual testing covering cluster creation using Blueprint and than
cluster upscaling.
Unit tests: Tests run: 4929, Failures: 0, Errors: 0, Skipped: 39
Thanks,
Sebastian Toader
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57039/#review166710
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 24, 2017, 3:19
://reviews.apache.org/r/56987/diff/
Testing
---
Performed manual testing covering cluster creation using Blueprint and than
cluster upscaling.
Unit tests: Tests run: 4929, Failures: 0, Errors: 0, Skipped: 39
Thanks,
Sebastian Toader
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56980/#review166524
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 23, 2017, 1:11
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56800/#review166179
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 17, 2017, 9:52
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56783/#review166155
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 21, 2017, 1:03
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56798/#review166059
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 17, 2017, 8:58
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56783/#review165939
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 17, 2017, 4:16
/package/scripts/params.py
(line 107)
<https://reviews.apache.org/r/56783/#comment237788>
If this list of znodes can not be retrieved from some kafka config than
specify here the link to the document that lists the kafka znodes
- Sebastian Toader
On Feb. 17, 2017, 3:54 p.m., Attila
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56766/#review165934
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 17, 2017, 2:09
ld add too much complexity and performance hit to the queries like
```ClusterConfigEntity.findEnabledConfigsByTypes``` ?
- Sebastian Toader
On Feb. 16, 2017, 10:12 p.m., Jonathan Hurley wrote:
>
> ---
> This is an automatically g
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56659/#review165725
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 15, 2017, 5:55
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56713/#review165718
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 15, 2017, 4:24
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56685/#review165714
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 15, 2017, 4:02
/configuration/Configuration.java
(line 513)
<https://reviews.apache.org/r/56685/#comment237523>
I searched for ```CVE-2014-3582``` on the web but couldn't find a detailed
description of this vulnerability. Should a direct link be listed here that
points to the detailed description.
- Sebas
upgraded from
2.4 to 2.5 works as expected after the upgrade.
- Sebastian Toader
On Feb. 14, 2017, 6:07 p.m., Eugene Chekanskiy wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56600/#review165324
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 13, 2017, 2:24
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56545/#review165235
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 10, 2017, 3:21
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56540/#review165119
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 10, 2017, 1:21
/DatabaseConsistencyCheckHelper.java
(line 657)
<https://reviews.apache.org/r/56540/#comment236927>
It would be useful to log the schema name that is first on the search path
as well.
- Sebastian Toader
On Feb. 10, 2017, 1:21 p.m., Balázs Bence Sári
> On Feb. 9, 2017, 6 p.m., Sebastian Toader wrote:
> > Did the manual testing covered both Ambari upgrade and HDP upgrade
> > scenarios to ensure that it works as expected? Also please include Jaimin
> > Jetly to the list of reviewers in order to review the UI part.
9
Thanks,
Sebastian Toader
tested the upgrading of HDP 2.4 to HDP 2.5 with "install_packages"
command failing on at least one agent.
Unit testing:
mvn test -DskipPythonTests -am -pl ambari-server
Results :
Tests run: 4906, Failures: 0, Errors: 0, Skipped: 39
Thanks,
Sebastian Toader
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56390/#review164569
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 7, 2017, 5:24
t;
HDFS ZKFC all capital.
ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/nonrolling-upgrade-2.6.xml
(line 329)
<https://reviews.apache.org/r/56331/#comment236240>
HDFS ZKFC all capital.
- Sebastian Toader
On Feb. 7, 2017, 10:02 a.m., A
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56331/#review164357
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 6, 2017, 5:26
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56179/#review164352
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 6, 2017, 3:59
For example the principals can be overriden through
custom kerberos descriptors thus this hard coded user list would not apply
anymore.
- Sebastian Toader
On Feb. 4, 2017, 6:31 p.m., Oliver Szabo wrote:
>
> ---
> This is an aut
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56282/#review164129
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 3, 2017, 6:13
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56225/#review164110
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 3, 2017, 11:39
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56179/#review163992
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 2, 2017, 5:23
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56225/#review163969
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 2, 2017, 2:36
ssion is given.
Authorisation enforced based only on the service name and not the fully
qualified principal could lead to security breach.
- Sebastian Toader
On Feb. 1, 2017, 9:46 p.m., Oliver Szabo wrote:
>
> ---
> This is an
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56175/#review163823
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 1, 2017, 5:09
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56173/#review163815
---
Ship it!
Ship It!
- Sebastian Toader
On Feb. 1, 2017, 4:11
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56133/#review163795
---
Ship it!
- Sebastian Toader
On Jan. 31, 2017, 7:49 p.m
How often will this log at INFO level?
ambari-agent/src/main/python/ambari_agent/Controller.py (line 477)
<https://reviews.apache.org/r/56133/#comment235301>
How often will this log on INFO level?
- Sebastian Toader
On Jan. 31, 2017, 7:49 p.m., Attila Doroszlai wrote:
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56022/#review163656
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 31, 2017, 2:43
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56076/#review163511
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 30, 2017, 2:04
]
[INFO] Ambari Server . SUCCESS [2:25.628s]
[INFO]
[INFO] BUILD SUCCESS
```
Thanks,
Sebastian Toader
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56054/#review163406
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 28, 2017, 6:16
--
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56022/
> ---
>
> (Updated Jan. 27, 2017, 7:24 p.m.)
>
>
> Review request for Ambari, La
Jan. 27, 2017, 7:24 p.m.)
>
>
> Review request for Ambari, Laszlo Puskas, Robert Levas, and Sebastian Toader.
>
>
> Bugs: AMBARI-19683
> https://issues.apache.org/jira/browse/AMBARI-19683
>
>
> Repository: ambari
>
>
> Description
> ---
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55987/#review163282
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 27, 2017, 5:12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56021/#review163267
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 27, 2017, 3:38
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55987/#review163142
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 26, 2017, 6:03
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55985/#review163136
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 26, 2017, 5:38
/package/scripts/params.py
(line 84)
<https://reviews.apache.org/r/55985/#comment234578>
Is it application-properties or atlas-application.properties
- Sebastian Toader
On Jan. 26, 2017, 5:38 p.m., Attila Magyar wrote:
>
> --
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55983/#review163123
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 26, 2017, 2:31
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55883/#review162823
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 24, 2017, 4:12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55882/#review162811
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 24, 2017, 3:16
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55833/#review162804
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 24, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55821/#review162626
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 22, 2017, 7:57
/stack_features.json
(line 26)
<https://reviews.apache.org/r/55833/#comment233972>
Maybe the description should be more explicit. eg. "Protect ZNodes with
SASL acl in secure clusters"
- Sebastian Toader
On Jan. 23, 2017, 10:29 a.m., Att
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55823/#review162605
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 23, 2017, 12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55822/#review162603
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 22, 2017, 10
/apache/ambari/logfeeder/util/SSLUtil.java
(line 129)
<https://reviews.apache.org/r/55821/#comment233970>
Isn't this an error that should be signaled wuth an Exception since the
password file is provided?
- Sebastian Toader
On Jan. 22, 2017, 7:57 p.m., Oliver
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55818/#review162566
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 21, 2017, 6:04
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55777/#review162467
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 20, 2017, 7:30
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55764/#review162432
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 20, 2017, 2:42
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55762/#review162418
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 20, 2017, 2:17
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55680/#review162417
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 20, 2017, 12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55643/#review162341
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 19, 2017, 8:49
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55724/#review162326
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 19, 2017, 6:59
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55680/#review162278
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 19, 2017, 1:39
/ambari/server/utils/TimeSource.java
(lines 1 - 30)
<https://reviews.apache.org/r/55643/#comment233595>
This abstraction doesn't seem to be necessary. You should be able to mock
System.currentTimeMillis() in tests.
- Sebastian Toader
On Jan. 17, 2017, 10:30 p.m., Balázs Bence
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55680/#review162153
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 18, 2017, 6:18
-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/utils.py
(line 392)
<https://reviews.apache.org/r/55680/#comment233389>
Not the ```cluster``` but the ```NameNode```
- Sebastian Toader
On Jan. 18, 2017, 4:58 p.m., Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55617/#review161872
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 17, 2017, 2:28
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55574/#review161779
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 16, 2017, 8:07
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55513/#review161749
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 13, 2017, 7:14
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55575/#review161732
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 16, 2017, 3:14
d and blueprint (NN
non-HA and HA).
Unit tests:
mvn test -Drat.skip=true -Dcheckstyle.skip -DskipSurefireTests
OK
--
Total run:1159
Total errors:0
Total failures:0
OK
Thanks,
Sebastian Toader
users to change
the ```principal_suffix``` later?
- Sebastian Toader
On Jan. 13, 2017, 7:14 p.m., Robert Levas wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
. SUCCESS [1:45.492s]
[INFO] Ambari Agent .. SUCCESS [25.860s]
[INFO]
[INFO] BUILD SUCCESS
Thanks,
Sebastian Toader
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55016/#review161356
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 11, 2017, 11
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55425/#review161246
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 11, 2017, 3:24
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55414/#review161207
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 11, 2017, 11
/controller/AmbariManagementControllerImpl.java
(line 242)
<https://reviews.apache.org/r/55342/#comment232308>
Document what "QuickLinksProfile" identify
- Sebastian Toader
On Jan. 9, 2017, 4:07 p.m., Balázs
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55016/#review160887
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 9, 2017, 11:51
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55250/#review160883
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 9, 2017, 11:53
checking that all agents
reconnect fine and continues executing status commands
2. Restarting agents
Unit tests:
---
Ran 452 tests in 107.184s
Thanks,
Sebastian Toader
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55282/#review160833
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 6, 2017, 10:59
and checking that all agents
reconnect fine and continues executing status commands
2. Restarting agents
Unit tests:
---
Ran 452 tests in 107.184s
Thanks,
Sebastian Toader
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55308/#review160832
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 7, 2017, 10:34
(verify that `agent.auto.cache.update=false` setting from server
is applied in status command executor child process).
ambari-agent unit tests:
--
Ran 452 tests in 101.627s
OK
Thanks,
Sebastian Toader
:
--
Ran 452 tests in 101.627s
OK
Thanks,
Sebastian Toader
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55178/#review160525
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 4, 2017, 5:56
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55171/#review160495
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 4, 2017, 4:10
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55171/#review160486
---
Ship it!
Ship It!
- Sebastian Toader
On Jan. 4, 2017, 1:42
tically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/55016/
> ---
>
> (Updated Dec. 23, 2016, 7:05 p.m.)
>
>
> Review request for Ambari, Andrew Onischuk, Balázs Bence Sári, Jonathan
> Hurley, Laszlo Puskas, an
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54891/#review160100
---
Ship it!
Ship It!
- Sebastian Toader
On Dec. 23, 2016, 8:54
201 - 300 of 546 matches
Mail list logo