---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61617/#review182894
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 14, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61619/#review182884
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 14, 2017, 3
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61024/#review182881
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 8, 2017, 5
/package/scripts/yarn.py
Lines 330 (patched)
<https://reviews.apache.org/r/61592/#comment258709>
Also change this in HDP 3.0
- Alejandro Fernandez
On Aug. 11, 2017, 9:07 a.m., Attila Doroszlai wrote:
>
> ---
> This is a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61527/#review182625
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 10, 2017, 9
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61542/#review182532
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 9, 2017, 9
/functions/get_stack_version.py
Line 88 (original), 88 (patched)
<https://reviews.apache.org/r/61525/#comment258420>
Just do this instead,
[0-9]+.[0-9]+.[0-9]+.[0-9]+(-[0-9]+)?
- Alejandro Fernandez
On Aug. 9, 2017, 1:07 p.m., Mugdha Varadkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61521/#review182506
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 9, 2017, 10
mmands/StackAdvisorCommand.java
Lines 184 (patched)
<https://reviews.apache.org/r/61024/#comment258334>
Why is this issuing an HTTP call instead of calling the method directly to
make it more efficient?
- Alejandro Fernandez
On Aug. 8, 2017, 5:57 p.m., Balázs Benc
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61500/#review182425
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 8, 2017, 4
/HDP/2.0.6/hooks/before-START/scripts/params.py
Line 334 (original), 334 (patched)
<https://reviews.apache.org/r/61481/#comment258204>
Also change in the hooks for HDP 3.0
- Alejandro Fernandez
On Aug. 7, 2017, 9:10 p.m., Sid Wagle
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61479/#review182316
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 7, 2017, 8
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61453/#review182306
---
What OS'es was this tested on?
- Alejandro Fernandez
On Aug. 7
/ambari/server/state/UpgradeContext.java
Lines 931 (patched)
<https://reviews.apache.org/r/61462/#comment258186>
Typo in orchestration
- Alejandro Fernandez
On Aug. 7, 2017, 3:04 p.m., Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61459/#review182286
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 7, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61456/#review182285
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 7, 2017, 11
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61447/#review182264
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 6, 2017, 11
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61414/#review182170
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 3, 2017, 8
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61415/#review182160
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 3, 2017, 8
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61405/#review182133
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 3, 2017, 1
283/diff/1-2/
Testing
---
Inspected manually
Thanks,
Alejandro Fernandez
61283/#comment257877>
ranger-knox-audit exists in both BigInsights 4.2 and 4.2.5 and did not
delete the properties in 4.2.5, so both upgrade packs need to delete these
configs.
- Alejandro Fernandez
On Aug. 1, 2017, 1
> On Aug. 2, 2017, 11:44 a.m., Tim Thorpe wrote:
> > Are these needed in BigInsights 4.2.5 as well?
>
> Alejandro Fernandez wrote:
> EU from BigInsights 4.2 to 4.2.5 already fixed most/all of these
> properties already.
I can still include it in 4.2.5 just to b
matically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61283/#review181964
---
On Aug. 1, 2017, 10:34 p.m., Alejandro Fernandez wrote:
>
> ---
> This is an automat
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61317/#review182035
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 2, 2017, 12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61376/#review182024
---
Ship it!
Ship It!
- Alejandro Fernandez
On Aug. 2, 2017, 10
b_user}}",
```
Diffs
-
ambari-server/src/main/resources/stacks/BigInsights/4.2/upgrades/config-upgrade.xml
17e80b5
ambari-server/src/main/resources/stacks/BigInsights/4.2/upgrades/nonrolling-upgrade-to-hdp-2.6.xml
55736cf
Diff: https://reviews.apache.org/r/61283/diff/1/
Testing
---
Inspected manually
Thanks,
Alejandro Fernandez
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61209/#review181908
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 28, 2017
/ambari/server/state/ConfigHelper.java
Lines (patched)
<https://reviews.apache.org/r/61266/#comment257614>
Never hit here
- Alejandro Fernandez
On July 31, 2017, 9:19 p.m., Nate Cole wrote:
>
> ---
> This is a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61239/#review181837
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 31, 2017, 6
> On July 31, 2017, 4:25 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
> > Lines 1123 (patched)
> > <https://reviews.apache.org/r/61239/diff/1/?file=178
gle digit, instead look for the
numeric characters from the end.
- Alejandro Fernandez
On July 29, 2017, 2:12 a.m., Di Li wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://revie
/internal/ClusterStackVersionResourceProvider.java
Lines 509 (patched)
<https://reviews.apache.org/r/61239/#comment257536>
Why is this assuming Power PC will exist?
- Alejandro Fernandez
On July 29, 2017, 2:12 a.m., Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61235/#review181750
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 28, 2017
> On July 27, 2017, 7:43 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/resources/stacks/BigInsights/4.2.5/upgrades/config-upgrade.xml
> > Lines 268 (patched)
> > <https://reviews.apache.org/r/61178/diff/1/?file=1784488#file1784488line268>
> >
&g
ardcoding /hdp/ext
- Alejandro Fernandez
On July 27, 2017, 4:09 p.m., Vitalyi Brodetskyi wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> http
/upgrades/config-upgrade.xml
Lines 268 (patched)
<https://reviews.apache.org/r/61178/#comment257191>
Is this deleting the config with the original value from the BigInsights
stack?
Would EU to HDP 2.6 create it with a new config value?
- Alejandro Fernandez
On July 27, 2017, 2:28 p.m.,
/ambari/server/serveraction/upgrades/FixAuthToLocalMappingAction.java
Lines 34 (patched)
<https://reviews.apache.org/r/61168/#comment257187>
Add some documentation as to why this class was needed and how it works.
- Alejandro Fernandez
On July 27, 2017, 2:04 a.m., Sid Wagle
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61132/#review181595
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 26, 2017, 1
ttps://reviews.apache.org/r/61129/diff/2/
Changes: https://reviews.apache.org/r/61129/diff/1-2/
Testing
---
Verified on a live cluster.
Thanks,
Alejandro Fernandez
.
ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/custom_extensions.py
Lines 38 (patched)
<https://reviews.apache.org/r/61107/#comment256961>
Why is iop hardcoded here?
- Alejandro Fernandez
On July 25, 2017, 11:29 p.m., Vitalyi Brodetskyi
> On July 25, 2017, 8:31 p.m., Alejandro Fernandez wrote:
> > ambari-server/src/main/resources/stacks/BigInsights/4.2.5/upgrades/config-upgrade.xml
> > Lines 75 (patched)
> > <https://reviews.apache.org/r/61102/diff/1/?file=1782056#file1782056line75>
> >
> &g
che.org/r/61102/#comment256913>
HDP 2.6 defaults to exectype=tez, so why was this needed?
- Alejandro Fernandez
On July 25, 2017, 2:11 p.m., Attila Doroszlai wrote:
>
> ---
> This is an automatically generated e-mail. To
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61105/#review181371
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 25, 2017
nerated e-mail. To reply, visit:
> https://reviews.apache.org/r/61083/
> ---
>
> (Updated July 24, 2017, 2:51 p.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jayush Luniya,
> Madhan Neethiraj, Mugdha Varadkar, Nixon
/UpgradeCatalog252.java
Lines 310 (patched)
<https://reviews.apache.org/r/61050/#comment256794>
MySQL shouldn't be restarted prior to the EU.
I would prefer to have this be in the EU config packs.
- Alejandro Fernandez
On July 21, 2017, 8:18 p.m., Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61036/#review181242
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 21, 2017, 3
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61079/#review181241
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 24, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61083/#review181238
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 24, 2017, 2
---
Python unit tests passed,
--
Total run:1161
Total errors:0
Total failures:0
OK
Thanks,
Alejandro Fernandez
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61006/#review181080
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 20, 2017, 8
ew cluster installs have 1024, and migrated clusters will
preserve their existing config value but have the python script fix it.
- Alejandro
---
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/61000/#review181071
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 20, 2017, 4
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60995/#review181067
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 20, 2017
ws.apache.org/r/60986/diff/2/
Changes: https://reviews.apache.org/r/60986/diff/1-2/
Testing
---
Python unit tests passed,
------
Total run:1161
Total errors:0
Total failures:0
OK
Thanks,
Alejandro Fernandez
/package/scripts/params_linux.py
Lines 73-74 (patched)
<https://reviews.apache.org/r/60986/#comment256405>
Ah, good catch, I wrote it too quickly
- Alejandro Fernandez
On July 19, 2017, 11:04 p.m., Alejandro Fernandez
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60982/#review180984
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 19, 2017, 8
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60945/#review180829
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 18, 2017, 5
write an existing custom value.
- Alejandro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60919/#review180732
-------
On July 17, 2017, 9:31 p.m., Alejandro Fernandez wrote:
>
> ---
use the if-present attributes?
Good catch, just fixed it.
- Alejandro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60919/#review180727
-------
Diff: https://reviews.apache.org/r/60919/diff/2/
Changes: https://reviews.apache.org/r/60919/diff/1-2/
Testing
---
Python unit tests passed.
--
Total run:1161
Total errors:0
Total failures:0
OK
Thanks,
Alejandro Fernandez
ges: https://reviews.apache.org/r/60886/diff/4-5/
Testing
---
Verified on local cluster. Unit test for UpgradeCatalog252Test.java passed.
Thanks,
Alejandro Fernandez
atched)
<https://reviews.apache.org/r/60919/#comment255945>
The id here was wrong, so just dropped the last 2 digits.
- Alejandro Fernandez
On July 17, 2017, 7:36 p.m., Alejandro Fernandez wrote:
>
> ---
> This is an auto
/
Testing
---
Python unit tests passed.
--
Total run:1161
Total errors:0
Total failures:0
OK
Thanks,
Alejandro Fernandez
>
> > Why indented?
Fixed
- Alejandro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60886/#review180710
---
On July 17, 201
ges: https://reviews.apache.org/r/60886/diff/3-4/
Testing
---
Verified on local cluster. Unit test for UpgradeCatalog252Test.java passed.
Thanks,
Alejandro Fernandez
------
On July 17, 2017, 7:12 p.m., Alejandro Fernandez wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60886/
>
ges: https://reviews.apache.org/r/60886/diff/2-3/
Testing
---
Verified on local cluster. Unit test for UpgradeCatalog252Test.java passed.
Thanks,
Alejandro Fernandez
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60916/#review180712
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 17, 2017, 5
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60912/#review180711
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 17, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60906/#review180705
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 17, 2017
/upgrades/nonrolling-upgrade-to-hdp-2.6.xml
Lines 945 (patched)
<https://reviews.apache.org/r/60916/#comment255929>
Why only one express upgrade pack and not both?
- Alejandro Fernandez
On July 17, 2017, 5:40 p.m., Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60859/#review180690
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 14, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60859/#review180689
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 14, 2017
.
- Alejandro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60886/#review180617
---
On July 15, 2017, 12:20 a.m., Alejandro
usters.
- Alejandro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60886/#review180681
---
On July 15, 2017, 12:20 a.m., Alej
tps://reviews.apache.org/r/60886/diff/2/
Changes: https://reviews.apache.org/r/60886/diff/1-2/
Testing
---
Verified on local cluster. Unit test for UpgradeCatalog252Test.java passed.
Thanks,
Alejandro Fernandez
t;
Will raise a Jira for Ambari 2.5.3 for the UI to select config types that
only contain configs with empty values.
- Alejandro Fernandez
On July 15, 2017, 12:14 a.m., Alejandro Fernandez wrote:
>
> ---
> This is an aut
est for UpgradeCatalog252Test.java passed.
Thanks,
Alejandro Fernandez
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60842/#review180559
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 14, 2017, 6
/resources/stacks/BigInsights/4.2.5/metainfo.xml
3cf364e
Diff: https://reviews.apache.org/r/60855/diff/1/
Testing
---
Verified on local cluster.
Thanks,
Alejandro Fernandez
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60850/#review180476
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 13, 2017, 8
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60849/#review180459
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 13, 2017, 7
re-run the
PreCheck and potentially find other failures.
- Alejandro Fernandez
On July 13, 2017, 8:01 p.m., Jonathan Hurley wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
nv.xml
Lines 32 (patched)
<https://reviews.apache.org/r/60819/#comment255528>
Added to all of the configs.
- Alejandro Fernandez
On July 12, 2017, 11:21 p.m., Alejandro Fernandez wrote:
>
> ---
> This is an automatic
ge/scripts/titan_client.py
PRE-CREATION
ambari-server/src/main/resources/stacks/BigInsights/4.2/upgrades/nonrolling-upgrade-to-hdp-2.6.xml
4867626
Diff: https://reviews.apache.org/r/60819/diff/1/
Testing
---
Verified on local cluster.
Thanks,
Alejandro Fernandez
: https://reviews.apache.org/r/60809/diff/2/
Changes: https://reviews.apache.org/r/60809/diff/1-2/
Testing
---
Verified on local cluster.
Thanks,
Alejandro Fernandez
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60807/#review180334
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 12, 2017, 4
che.org/r/60809/diff/1/
Testing
---
Verified on local cluster.
Thanks,
Alejandro Fernandez
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60802/#review180311
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 12, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60797/#review180310
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 12, 2017, 9
/configuration/hive-site.xml
Lines 19 (patched)
<https://reviews.apache.org/r/60782/#comment255403>
This is also needed in Hive for 3.0
- Alejandro Fernandez
On July 11, 2017, 7:51 p.m., Vitalyi Brodetskyi
reErrors=true
# Verify the contents
rpm -qlp rpm -qlp ambari-server-2.5.1.0-0.x86_64.rpm | grep
"var/lib/ambari-server/resources/stacks/BigInsights"
rpm -qlp rpm -qlp ambari-agent-2.5.1.0-0.x86_64.rpm | grep
"/var/lib/ambari-agent/cache/stacks/BigInsights"
```
Thanks,
Alejandro Fernandez
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60754/#review180115
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 10, 2017, 9
/0.96.0.2.0/package/scripts/params_linux.py
Lines 445 (patched)
<https://reviews.apache.org/r/60691/#comment255074>
Lines 445 and 446 are no longer needed now.
- Alejandro Fernandez
On July 10, 2017, 3:54 p.m., Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60744/#review180062
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 10, 2017, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60745/#review180060
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 10, 2017, 2
/configuration/cluster-env.xml
Lines 211 (patched)
<https://reviews.apache.org/r/60741/#comment255067>
We should actually set all of these to on-ambari-upgrade add="false" to
minimize the risk of new properties being added before the EU migration.
- Alejandro Fernandez
On July 10,
/0.96.0.2.0/package/scripts/hbase.py
Lines 197 (patched)
<https://reviews.apache.org/r/60738/#comment255062>
This is already needed in HBase for 3.0
Please add a comment as well explaining why this was needed.
- Alejandro Fernandez
On July 10, 2017, 9:47 a.m., Andrew Onischuk
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60709/#review179956
---
Ship it!
Ship It!
- Alejandro Fernandez
On July 7, 2017, 9
1 - 100 of 1418 matches
Mail list logo