from the community.
https://cwiki.apache.org/confluence/display/AMBARI/How+to+Contribute
- Jonathan Hurley
On Feb. 14, 2018, 10:25 a.m., Selvamohan Neethiraj wrote:
>
> ---
> This is an automatically generated e-mail. To rep
---
PENDING
Thanks,
Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64950/#review194766
---
Ship it!
Ship It!
- Jonathan Hurley
On Jan. 4, 2018, 9:44
/ZEPPELIN/0.7.0/package/scripts/master.py
Lines 397 (patched)
<https://reviews.apache.org/r/64894/#comment273559>
Maybe just comment on why this is needed ...
- Jonathan Hurley
On Jan. 2, 2018, 2:18 p.m., Prabhjyot Singh
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64853/#review194609
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 27, 2017, 10:58
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64815/#review194458
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 22, 2017, 4:43
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64785/#review194426
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 21, 2017, 4 p.m
fs-mpack/src/main/resources/addon-services/ONEFS/1.0.0/package/scripts/params_linux.py
Line 21 (original), 20 (patched)
<https://reviews.apache.org/r/64772/#comment273122>
Remove import * in favor of explicit imports.
- Jonathan Hurley
On Dec. 21, 2017, 9:46 a.m., A
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64751/#review194333
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 21, 2017, 8:25
c/main/resources/stacks/HDP/2.0.6/properties/stack_packages.json
Lines 1075-1080 (original), 1090-1096 (patched)
<https://reviews.apache.org/r/64751/#comment272978>
nifi is supported by hdp-select?
- Jonathan Hurley
On Dec. 20, 2017, 9:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64722/#review194185
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 19, 2017, 3:58
> On Dec. 17, 2017, 2:14 p.m., Jonathan Hurley wrote:
> > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py
> > Lines 155 (patched)
> > <https://reviews.apache.org/r/64667/diff/1/?file=1919066#file1919066line156>
> >
> > OK -
> On Dec. 17, 2017, 2:14 p.m., Jonathan Hurley wrote:
> > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py
> > Lines 155 (patched)
> > <https://reviews.apache.org/r/64667/diff/1/?file=1919066#file1919066line156>
> >
> > OK -
ix_default_links"
We should also have some documentation as to why we need this method ...
- Jonathan Hurley
On Dec. 16, 2017, 10:53 a.m., Dmitro Lisnichenko wrote:
>
> ---
> This is an automatically generated e-
> On Dec. 16, 2017, 3:35 p.m., Jonathan Hurley wrote:
> > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py
> > Lines 160-164 (patched)
> > <https://reviews.apache.org/r/64667/diff/1/?file=1919066#file1919066line161>
> >
> >
> On Dec. 16, 2017, 3:35 p.m., Jonathan Hurley wrote:
> > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py
> > Lines 160-164 (patched)
> > <https://reviews.apache.org/r/64667/diff/1/?file=1919066#file1919066line161>
> >
> >
/install_packages.py
Lines 160-164 (patched)
<https://reviews.apache.org/r/64667/#comment272709>
We should not hard code this into install_packages. Can we just expose this
method for the Livy Python files to call?
- Jonathan Hurley
On Dec. 16, 2017, 10:53 a.m., Dmitro Lisnichenko
; > These records should just be straight up deleted, not marked this way.
>
> Myroslav Papirkovskyy wrote:
> UI makes explicit call to delete host versions, which will fail if we
> just remove records.
>
> Jonathan Hurley wrote:
> If the UI is deleting the records
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64652/#review193959
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 15, 2017, noon
On Dec. 15, 2017, noon, Myroslav Papirkovskyy wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64652/
> ---
>
> (Updated Dec. 15, 2017, noon)
>
>
> Review request for
deleting the
associated cluster. There is an association between repo, host_version, and
cluster, so we can target only those which are scoped to the correct cluster.
- Jonathan Hurley
On Dec. 15, 2017, noon, Myroslav Papirkovskyy wrote
> On Dec. 12, 2017, 11:44 a.m., Jonathan Hurley wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog300.java
> > Lines 175-192 (patched)
> > <https://reviews.apache.org/r/64544/diff/1/?file=1914302#file1914302line175>
> &g
-
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64637/#review193874
-------
On Dec. 14, 2017, 10:03 p.m., Jonathan Hurley wrote:
>
>
FO] Finished at: 2017-12-14T22:37:14-05:00
[INFO] Final Memory: 21M/619M
[INFO] ----
Thanks,
Jonathan Hurley
ly verified PID detection on non-root cluster.
Thanks,
Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64537/#review193731
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 13, 2017, 10:42
the ATS machine.
Diffs
-
ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py
ac71ce4b36
Diff: https://reviews.apache.org/r/64579/diff/1/
Testing
---
Manual testing on a patched cluster with YARN/Spark
Thanks,
Jonathan Hurley
s/hbase.py
ac71ce4b36
Diff: https://reviews.apache.org/r/64554/diff/2/
Changes: https://reviews.apache.org/r/64554/diff/1-2/
Testing
---
Manual testing on Hbase cluster.
Thanks,
Jonathan Hurley
eviews.apache.org/r/64554/#review193680
---
On Dec. 13, 2017, 9:02 a.m., Jonathan Hurley wrote:
>
> ---
> This is an automatically generated e-mail. To re
554/diff/1/
Testing
---
Manual testing on Hbase cluster.
Thanks,
Jonathan Hurley
ranger
function and just invoke it?
Also - shouldn't ranger just ship with a default core-site.xml in these cases?
Seems kind of odd to have to create a dummy one...
- Jonathan Hurley
On Dec. 12, 2017, 7:35 a.m., Vishal Suvagia
t;
You should create the FK declarations inside the table declarations.
- Jonathan Hurley
On Dec. 12, 2017, 9:35 a.m., Robert Levas wrote:
>
> ---
> This is an automatically generated e-mail. To reply, vi
iff/2-3/
Testing
---
Manual install via UI and Blueprint
Thanks,
Jonathan Hurley
version if it's not trusted, as is the
case with "latest" installations.
- Jonathan Hurley
On Dec. 11, 2017, 12:04 p.m., Jonathan Hurley wrote:
>
> ---
> This is an automatically generated e-mail. To reply, vis
er fill in the
correct values.
Diffs
-
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
98141456c7
Diff: https://reviews.apache.org/r/64502/diff/1/
Testing
---
Manual install via UI and Blueprint
Thanks,
Jonathan Hurley
4419/diff/1-2/
Testing
---
Manual upgrade testing.
Thanks,
Jonathan Hurley
pache.org/r/64419/diff/1/
Testing
---
Manual upgrade testing.
Thanks,
Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64388/#review193124
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 7, 2017, 9:43
/AmbariManagementControllerImpl.java
Lines 3092-3096 (patched)
<https://reviews.apache.org/r/64388/#comment271624>
One more question - what is the reason for this code? Is it necessary for
this fix?
- Jonathan Hurley
On Dec. 7, 2017, 9:43 a.m., Myroslav Papirkovskyy
> On Dec. 6, 2017, 2 p.m., Jonathan Hurley wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/HostVersionOutOfSyncListener.java
> > Lines 134-139 (patched)
> > <https://reviews.apache.org/r/64388/diff/1/?file=1909709#file1909709line
the other method onServiceEvent(...)
- Jonathan Hurley
On Dec. 6, 2017, 1:52 p.m., Myroslav Papirkovskyy wrote:
>
> ---
> This is an automatically generated e-mail. To reply, vis
g stack is being used to
lookup the information. HDP should be used since it's current...
- Jonathan Hurley
On Dec. 6, 2017, 1:52 p.m., Myroslav Papirkovskyy wrote:
>
> ---
> This is an automatically generated e-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64261/#review192984
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 1, 2017, 4:53
/upgrade/UpgradeCatalog261.java
Lines 161 (patched)
<https://reviews.apache.org/r/64311/#comment271243>
You can use LOG.debug's built in support of {}
- Jonathan Hurley
On Dec. 5, 2017, 12:39 p.m., Di Li wrote:
>
> ---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64346/#review192896
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 5, 2017, 12:46
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64346/#review192897
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 5, 2017, 12:46
]
[INFO] Total time: 01:29 min
[INFO] Finished at: 2017-12-05T13:27:57-05:00
[INFO] Final Memory: 21M/619M
[INFO]
Thanks,
Jonathan Hurley
Lines 72 (patched)
<https://reviews.apache.org/r/64315/#comment271216>
Maybe also logger.info "This stack does not indicate that it supports LZO
installation"
- Jonathan Hurley
On Dec. 5, 2017, 11:51 a.m., Sid Wagle wrote:
>
> -
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64340/#review192881
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 5, 2017, 10:04
/UpgradeCatalog261.java
Lines 154 (patched)
<https://reviews.apache.org/r/64311/#comment271178>
We really shouldn't be hardcoding stack names and versions like this.
what change caused this problem? there must be a better workaround
- Jonathan Hurley
On Dec. 4, 2017, 3:08
org/apache/ambari/server/state/repository/VersionDefinitionTest.java
9fe6146a9c
Diff: https://reviews.apache.org/r/64316/diff/1/
Testing
---
PENDING
Thanks,
Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64315/#review192776
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 4, 2017, 3:33
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64310/#review192748
---
Ship it!
Ship It!
- Jonathan Hurley
On Dec. 4, 2017, 2:14
ache.org/r/64308/diff/1/
Testing
---
Manual
Thanks,
Jonathan Hurley
11-30T15:16:50-05:00
[INFO] Final Memory: 21M/619M
[INFO] --------
Thanks,
Jonathan Hurley
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64223/#review192357
---
On Nov. 30, 2017, 3:28 p.m., Jonathan Hurley wrote:
>
> ---
> This is an automatically generated e-mail. To reply, vi
[INFO] BUILD SUCCESS
[INFO]
[INFO] Total time: 58.251 s
[INFO] Finished at: 2017-11-30T15:16:50-05:00
[INFO] Final Memory: 21M/619M
[INFO] --
Lines 89 (patched)
<https://reviews.apache.org/r/64216/#comment270403>
Maybe some documentation so people know what a repo name is. Something
like: Gets the name of the repository without any version identifiers
- Jonathan Hurley
On Nov. 30, 2017, 12:14 p.m., Myroslav Papirk
/HBASE/package/scripts/params.py
Lines 24 (patched)
<https://reviews.apache.org/r/64205/#comment270364>
import * has gotten us into a lot of trouble. It's best to just remove it
and fill in the imports for this file explicitely...
- Jonathan Hurley
On Nov. 30, 2017, 10:04 a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64186/#review192237
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 29, 2017, 7:20
/ambari/server/actionmanager/ExecutionCommandWrapper.java
Lines 303-304 (original), 303-304 (patched)
<https://reviews.apache.org/r/64173/#comment270211>
Adjust documention to indicate why !resolved() is necessary.
- Jonathan Hurley
On Nov. 29, 2017, 3 p.m., Attila Doroszlai
visit:
https://reviews.apache.org/r/64166/#review192153
-------
On Nov. 29, 2017, 11:55 a.m., Jonathan Hurley wrote:
>
> ---
> This is an automatically generat
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64167/#review192138
---
Ship it!
No more strings! :)
- Jonathan Hurley
On Nov. 29
Diffs
-
ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py
bd1cede834
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
2871dc0ff3
Diff: https://reviews.apache.org/r/64166/diff/1/
Testing
---
Manual install & upgrade
Thanks,
Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64164/#review192130
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 29, 2017, 11:35
/
Testing
---
Manual
Thanks,
Jonathan Hurley
:00
[INFO] Final Memory: 18M/491M
[INFO]
Thanks,
Jonathan Hurley
with a page saying the UI was disabled.
Thanks,
Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64089/#review192033
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 27, 2017, 3:52
INFO]
[INFO] Total time: 59.118 s
[INFO] Finished at: 2017-11-27T13:47:56-05:00
[INFO] Final Memory: 21M/619M
[INFO]
Thanks,
Jonathan Hurley
properties/stack_packages.json
7a12011674
ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_packages.json
245449c535
Diff: https://reviews.apache.org/r/64085/diff/1/
Testing
---
Manual
Thanks,
Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64043/#review191847
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 22, 2017, 7:55
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63961/#review191735
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 21, 2017, 12:56
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64030/#review191734
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 22, 2017, 9 a.m
t ... and it also passed on some continuous
integration tests. But I'm fine moving it to sudo ...
- Jonathan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63962/#review191550
-----
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63982/#review191600
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 21, 2017, 5:58
/1/
Testing
---
Manual testing.
Thanks,
Jonathan Hurley
<https://reviews.apache.org/r/63955/#comment269374>
add missing throws to documentation
- Jonathan Hurley
On Nov. 20, 2017, 12:06 p.m., Myroslav Papirkovskyy wrote:
>
> ---
> This is an automatically generated e-mail. To
<https://reviews.apache.org/r/63957/#comment269370>
Documentation.
ambari-server/src/main/resources/stacks/HDP/2.0.6/services/stack_advisor.py
Lines 519 (patched)
<https://reviews.apache.org/r/63957/#comment269371>
Documentation.
- Jonathan Hurley
On Nov. 20, 2017, 12
--
>
> (Updated Nov. 17, 2017, 9:40 a.m.)
>
>
> Review request for Ambari, Attila Doroszlai, Jonathan Hurley, Myroslav
> Papirkovskyy, Nate Cole, and Sid Wagle.
>
>
> Bugs: AMBARI-22467
> https://issues.apache.org/jira/browse/AMBARI-22467
>
>
> Re
, do we think this should be a Fail?
ambari-common/src/main/python/resource_management/libraries/functions/get_lzo_packages.py
Lines 73 (patched)
<https://reviews.apache.org/r/63916/#comment269130>
Doc.
- Jonathan Hurley
On Nov. 17, 2017, 9:40 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63880/#review191207
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 16, 2017, 10:45
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63805/#review191071
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 14, 2017, 2:47
mon/src/main/python/resource_management/libraries/functions/tar_archive.py
c682c3e24a
Diff: https://reviews.apache.org/r/63807/diff/2/
Changes: https://reviews.apache.org/r/63807/diff/1-2/
Testing
---
Manual testing to ensure the file is created and uploaded.
UNIT TESTS PENDING...
Thank
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63822/#review191059
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 14, 2017, 8:57
es/functions/tar_archive.py
c682c3e24a
Diff: https://reviews.apache.org/r/63807/diff/1/
Testing
---
Manual testing to ensure the file is created and uploaded.
UNIT TESTS PENDING...
Thanks,
Jonathan Hurley
(patched)
<https://reviews.apache.org/r/63800/#comment268557>
Should this extend the configure task instead?
- Jonathan Hurley
On Nov. 14, 2017, 11:46 a.m., Di Li wrote:
>
> ---
> This is an automatically generated e-mail.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63725/#review190750
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 10, 2017, 10:26
/stack_packages.json
Line 1280 (original), 1280 (patched)
<https://reviews.apache.org/r/63713/#comment268233>
I think Tez can go by itself...
- Jonathan Hurley
On Nov. 9, 2017, 8:43 p.m., Nate Cole wrote:
>
> ---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63713/#review190670
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 9, 2017, 8:43
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63516/#review190668
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 9, 2017, 4:58
at that point...
- Jonathan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63684/#review190616
---
On Nov. 8, 2017, 4:2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63695/#review190579
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 9, 2017, 5:15
/
Testing
---
Manual upgrade on patched cluster.
Thanks,
Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63671/#review190467
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 8, 2017, 12:08
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63640/#review190466
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 7, 2017, 12:23
onary...
- Jonathan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63595/#review190288
---
On Nov. 6, 2017, 4:17 p.m., Jonathan Hurley
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63593/#review190221
---
Ship it!
Ship It!
- Jonathan Hurley
On Nov. 6, 2017, 3:31
--
Total run:1193
Total errors:0
Total failures:0
OK
Thanks,
Jonathan Hurley
e.org/r/63535/#comment267278>
Do you need to adjust the stack_packages for BigInsights as well?
Also - when you cherry pick, don't forget that there is a stack packages
under HDP 3.0 in trunk...
- Jonathan Hurl
1 - 100 of 1558 matches
Mail list logo