i-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog240.java
<https://reviews.apache.org/r/50594/#comment210435>
Same as above
- Jayush Luniya
On Aug. 1, 2016, 1:22 a.m., Saisai Shao wrote:
>
> ---
&g
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50656/#review144371
---
Ship it!
Ship It!
- Jayush Luniya
On Aug. 1, 2016, 12:22
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50512/#review143765
---
Ship it!
Ship It!
- Jayush Luniya
On July 27, 2016, 5:10
view143535
---
On July 27, 2016, 9:06 a.m., Mugdha Varadkar wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/5
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50445/#review143746
---
Ship it!
Ship It!
- Jayush Luniya
On July 27, 2016, 9:06
/package/scripts/params.py
(line 325)
<https://reviews.apache.org/r/50445/#comment209675>
Merge the 2 if-blocks
- Jayush Luniya
On July 27, 2016, 9:06 a.m., Mugdha Varadkar wrote:
>
> ---
> This is an automatically g
/stack_advisor.py (line
1686)
<https://reviews.apache.org/r/50445/#comment209538>
We should be consistent in naming conventions for variable names.
- Jayush Luniya
On July 26, 2016, 2:49 p.m., Mugdha Varadkar
/kerberos.json
(line 189)
<https://reviews.apache.org/r/50445/#comment209537>
How did this typo not cause problems in the past?
- Jayush Luniya
On July 26, 2016, 2:49 p.m., Mugdha Varadkar wrote:
>
> ---
> This is a
ot be Kerberized if Kerberos is enabled for the cluster?
> >
> > It seems like this is redundant info and possible problematic if a user
> > changes the value via Ranger's service config page or the REST API.
>
> Jayush Luniya wrote:
> Agree
scenario?
- Jayush Luniya
On July 26, 2016, 2:49 p.m., Mugdha Varadkar wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
--
>
> (Updated July 26, 2016, 2:49 p.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jaimin Jetly,
> Jayush Luniya, Robert Levas, Srimanth Gunturi, and Velmurugan Periasamy.
>
>
> Bug
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50394/#review143394
---
Ship it!
Ship It!
- Jayush Luniya
On July 25, 2016, 5:03
he way to Finalize step and kick
DOWNGRADE will old alerts be restored?
- Jayush Luniya
On July 21, 2016, 12:36 p.m., Gautam Borad wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
> On July 21, 2016, 3:30 p.m., Jonathan Hurley wrote:
> > It would have been nice to break this type of server action out into
> > something more generic for alert updates. That way, future changes like
> > this won't need to worry about broadcasting events. However, it's not
> > necessary
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50287/#review143111
---
Ship it!
Ship It!
- Jayush Luniya
On July 21, 2016, 10:19
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50300/#review143109
---
Ship it!
Ship It!
- Jayush Luniya
On July 21, 2016, 4:14
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50294/#review143101
---
Ship it!
Ship It!
- Jayush Luniya
On July 21, 2016, 2:15
> On July 20, 2016, 3:49 p.m., Jayush Luniya wrote:
> > Ship It!
@Rob, whats the Apache JIRA for this? Can you update the review with the Apache
JIRA?
- Jayush
---
This is an automatically generated e-mail. To reply, vis
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50237/#review142951
---
Ship it!
Ship It!
- Jayush Luniya
On July 20, 2016, 3:21
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50234/#review142950
---
Ship it!
Ship It!
- Jayush Luniya
On July 20, 2016, 2:09
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50196/#review142824
---
Ship it!
Ship It!
- Jayush Luniya
On July 19, 2016, 6:36
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/50087/
> -----------
>
> (Updated July 18, 2016, 8:51 p.m.)
>
>
> Review request for Ambari, Alexander Denissov, Jayush Luniya,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49987/#review142413
---
Ship it!
Ship It!
- Jayush Luniya
On July 14, 2016, 2:23
> On June 17, 2016, 3:04 p.m., Jayush Luniya wrote:
> > @Saisai
> > Looks like you need to rebase again
> > $ git apply ~/Downloads/AMBARI-16864-v1.patch
> > /Users/jluniya/Downloads/AMBARI-16864-v1.patch:1100: trailing whitespace.
> > "spark-javaop
> On June 21, 2016, 3:22 a.m., Jayush Luniya wrote:
> > Ship It!
Saisai,
Can you close this review?
- Jayush
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48999/#rev
> On July 15, 2016, 5:25 p.m., Jayush Luniya wrote:
> > Ship It!
Committed patch. Please close review.
- Jayush
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50030/#rev
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50030/#review142401
---
Ship it!
Ship It!
- Jayush Luniya
On July 15, 2016, 8:43
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49987/#review142266
---
Can we add any unit tests to cover these cases?
- Jayush Luniya
> On July 13, 2016, 12:30 a.m., Jonathan Hurley wrote:
> > I'm going to +1 this, but I'd still like to know how other commiters feel
> > about making the socket commands paramters vs actual parts of the source
> > element.
Parameters make more sense for script alerts than port alerts in my
> On July 11, 2016, 9:51 p.m., Masahiro Tanaka wrote:
> > Should I UpgradeCatalog250.java as we are readying to release Ambari2.4.0?
>
> Jonathan Hurley wrote:
> That depends; there's no Fix Version on this Jira, so I don't know if
> it's only going into trunk or if it's going to into
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49831/#review141725
---
Ship it!
Ship It!
- Jayush Luniya
On July 8, 2016, 9:13 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49328/#review141290
---
Ship it!
Ship It!
- Jayush Luniya
On July 8, 2016, 2:43 p.m
> On July 7, 2016, 7:32 a.m., Jayush Luniya wrote:
> > Ship It!
Committed
- Jayush
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49449/#rev
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49449/#review141140
---
Ship it!
Ship It!
- Jayush Luniya
On July 1, 2016, 5:56 a.m
/configuration/spark-defaults.xml
(line 1)
<https://reviews.apache.org/r/49295/#comment206177>
The self detection mechanism for hdp.version was added in HDP-2.3.4. So
shouldnt be remove these properties from HDP-2.4 onwards instead of HDP-2.5
onwards?
- Jayush Luniya
On June 29, 2016
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49429/#review140825
---
Ship it!
Ship It!
- Jayush Luniya
On June 30, 2016, 11:48
/serverConfiguration.py (line 187)
<https://reviews.apache.org/r/49328/#comment206160>
Aldo add property to ambari.properties
- Jayush Luniya
On June 30, 2016, 5:16 p.m., Tim Thorpe wrote:
>
> ---
> This is an automatically generated e
could have service_name : "ACCUMULO" so that the list of templates can be
filtered based on service_names in the stack definition.
Long term we should definitely push this down to the service-level. We wont be
able to support logsearch for custom services.
- Jayush Luniya
On June 30, 2
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49134/#review139323
---
Ship it!
Ship It!
- Jayush Luniya
On June 23, 2016, 6:18
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48970/#review138863
---
Ship it!
Ship It!
- Jayush Luniya
On June 20, 2016, 11:19
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47656/#review138862
---
Ship it!
Ship It!
- Jayush Luniya
On June 20, 2016, 4:18
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48999/#review138755
---
Ship it!
Ship It!
- Jayush Luniya
On June 21, 2016, 3:14
tps://reviews.apache.org/r/48734/#comment203429>
Not sure if stopping namenodes is the right way to go about with this.
- Jayush Luniya
On June 15, 2016, 4:41 p.m., Victor Galgo wrote:
>
> ---
> This is an automatically generated e-mail.
> On June 17, 2016, 2:57 p.m., Jayush Luniya wrote:
> > Ship It!
>
> Jayush Luniya wrote:
> Update patch in CR with
> https://issues.apache.org/jira/secure/attachment/12811383/AMBARI-16920-7.patch
Also close CR
> On June 15, 2016, 4:57 p.m., Jayush Luniya wrote:
> > Ship It!
@Jeff can you close this CR as the patch is committed?
- Jayush
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.or
trailing whitespace.
"hooks_folder": "HDP/2.0.6/hooks",
error: patch failed:
ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/spark_service.py:58
error:
ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/spark_servi
> On June 17, 2016, 2:57 p.m., Jayush Luniya wrote:
> > Ship It!
Update patch in CR with
https://issues.apache.org/jira/secure/attachment/12811383/AMBARI-16920-7.patch
- Jayush
---
This is an automatically generated e-mail.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47941/#review138226
---
Ship it!
Ship It!
- Jayush Luniya
On June 16, 2016, 11:06
> On June 16, 2016, 6:15 p.m., Jayush Luniya wrote:
> > This is causing unit test failures. Please take a look.
> >
> > ---
> > Failed tests:
> > FAIL: test_hook_default_conf_select
> > (
> On June 16, 2016, 6:15 p.m., Jayush Luniya wrote:
> > This is causing unit test failures. Please take a look.
> >
> > ---
> > Failed tests:
> > FAIL: test_hook_default_conf_select
> > (
> On June 2, 2016, 5:01 p.m., Jayush Luniya wrote:
> > Ship It!
@Jeff can you close this CR?
- Jayush
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46727/#rev
-
[INFO] Total time: 1:15.808s
[INFO] Finished at: Thu Jun 16 09:56:48 PDT 2016
[INFO] Final Memory: 60M/837M
[INFO]
. There are conflicts. Will commit once
I have the latest patch.
- Jayush Luniya
On June 13, 2016, 10:45 a.m., Jeff Zhang wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
> On June 2, 2016, 4:42 p.m., Jayush Luniya wrote:
> > ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/params.py,
> > line 1
> > <https://reviews.apache.org/r/47941/diff/1/?file=1396057#file1396057line1>
> >
> > Pleas
> On June 15, 2016, 5:51 p.m., Jayush Luniya wrote:
> > Ship It!
For 2.4 and trunk, we should not have any hdp-select and hdp hardcodings.
Everything should be stack config driven.
- Jayush
---
This is an automatically ge
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48308/#review137771
---
Ship it!
Ship It!
- Jayush Luniya
On June 14, 2016, 4:10
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48335/#review137770
---
Ship it!
Ship It!
- Jayush Luniya
On June 7, 2016, 11:45
)
<https://reviews.apache.org/r/48636/#comment202960>
fake?
- Jayush Luniya
On June 13, 2016, 1:22 p.m., Renjith Kamath wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
> On June 1, 2016, 8:55 p.m., Jayush Luniya wrote:
> > Ship It!
Lav, can you close the review if the patch is already committed?
- Jayush
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apa
ing whitespace.
"hooks_folder": "HDP/2.0.6/hooks",
error: patch failed:
ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/params.py:156
error:
ambari-server/src/main/resources/common-services/SPARK2/2.0.0/package/scripts/params.py:
patch does
> On June 15, 2016, 4:40 p.m., Jayush Luniya wrote:
> > Ship It!
Committed patch. Please close CR
- Jayush
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48659/#rev
> On June 10, 2016, 9:35 p.m., Jayush Luniya wrote:
> > Ship It!
>
> Masahiro Tanaka wrote:
> Thank you! Could you commit this ?
Committed, thanks for your contribution. Can you close the
16, 4:09 p.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Matt.
>
>
> Bugs: AMBARI-16916
> https://issues.apache.org/jira/browse/AMBARI-16916
>
>
> Repository: ambari
>
>
> Description
> ---
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48184/#review136845
---
Ship it!
Ship It!
- Jayush Luniya
On June 7, 2016, 4:09 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48332/#review136499
---
Ship it!
Ship It!
- Jayush Luniya
On June 7, 2016, 10:05
> On June 3, 2016, 5:16 p.m., Jayush Luniya wrote:
> > ambari-server/src/main/resources/stacks/HDP/2.5/services/SPARK/configuration/livy-conf.xml,
> > line 32
> > <https://reviews.apache.org/r/48212/diff/1-2/?file=1405757#file1405757line32>
> >
> &g
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48212/#review136366
---
Ship it!
Ship It!
- Jayush Luniya
On June 3, 2016, 4:55 p.m
de add="false" v/s on-ambari-upgrade add="true") Can you provide
examples?
I am not clear on why on-ambari-upgrade delete="true" and on-stack-upgrade
delete="false"?
I will look at your original patch to get some context and then review
la
tomatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48157/#review135933
-------
On June 2, 2016, 6:15 p.m., Jayush Luniya wrote:
>
> ---
> This is an au
y: 38M/1011M
[INFO] ----
Thanks,
Jayush Luniya
-mail. To reply, visit:
> https://reviews.apache.org/r/47579/
> ---
>
> (Updated May 23, 2016, 2:55 a.m.)
>
>
> Review request for Ambari, Jayush Luniya and Sumit Mohanty.
>
>
> Bugs: AMBARI-16753
> http
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47885/#review135972
---
Ship it!
Ship It!
- Jayush Luniya
On May 27, 2016, 4:30 a.m
> On June 2, 2016, 4:52 p.m., Jayush Luniya wrote:
> > ambari-server/src/main/resources/stacks/HDP/2.5/services/SPARK/kerberos.json,
> > line 1
> > <https://reviews.apache.org/r/46727/diff/18/?file=1404615#file1404615line1>
> >
> > Patch does not
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46727/#review135966
---
Ship it!
Ship It!
- Jayush Luniya
On June 2, 2016, 10:27
/kerberos.json
(line 1)
<https://reviews.apache.org/r/46727/#comment200974>
Patch does not apply. I dont see a kerberos.json file in HDP-2.5
ls ambari-server/src/main/resources/stacks/HDP/2.5/services/SPARK
metainfo.xml
- Jayush Luniya
On June 2, 2016, 10:27 a.m., Jeff
g/r/47941/#comment200972>
Use explicit imports
- Jayush Luniya
On May 27, 2016, 3:33 a.m., Jeff Zhang wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://re
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47653/#review135952
---
Ship it!
Ship It!
- Jayush Luniya
On May 20, 2016, 6:14 p.m
e path defined in one place
- Jayush Luniya
On May 27, 2016, 3:33 a.m., Jeff Zhang wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47923/#review135942
---
Ship it!
Ship It!
- Jayush Luniya
On May 26, 2016, 11:04
change in a new
CR request. I will verify it and commit this delta patch instead of reverting
the old patch.
- Jayush Luniya
On June 1, 2016, 11:06 p.m., Jeff Zhang wrote:
>
> ---
> This is an automatically generated e-mail. To rep
Memory: 38M/1011M
[INFO] ----
Thanks,
Jayush Luniya
y: 38M/1011M
[INFO] ----
Thanks,
Jayush Luniya
/test_ru_set_all.py 9a4dd31
ambari-server/src/test/python/stacks/2.2/common/test_conf_select.py 7fbda2d
ambari-server/src/test/python/stacks/utils/RMFTestCase.py 74454e5
Diff: https://reviews.apache.org/r/48157/diff/
Testing
---
Thanks,
Jayush Luniya
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47858/#review135854
---
Ship it!
Ship It!
- Jayush Luniya
On May 27, 2016, 7:11 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48143/#review135851
---
Ship it!
Ship It!
- Jayush Luniya
On June 1, 2016, 6:26 p.m
/package/scripts/nodemanager.py
(line 90)
<https://reviews.apache.org/r/47963/#comment200862>
@Juanjo, it looks like check_process_status is not imported for YARN
nodemanager and will cause problems. Can you submit an addendum patch with this
change?
- Jayush Luniya
On May 27, 2016
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47964/#review135757
---
Ship it!
Ship It!
- Jayush Luniya
On May 27, 2016, 4:36 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47963/#review135756
---
Ship it!
Ship It!
- Jayush Luniya
On May 27, 2016, 4:33 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47962/#review135755
---
Ship it!
Ship It!
- Jayush Luniya
On May 27, 2016, 4:28 p.m
-
>
> (Updated May 26, 2016, 11:04 p.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Matt.
>
>
> Bugs: AMBARI-16797
> https://issues.apache.org/jira/browse/AMBARI-16797
>
>
> Repository: ambari
>
>
> Description
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47934/#review135752
---
Ship it!
Ship It!
- Jayush Luniya
On May 27, 2016, 12:20
conflicts?
- Jayush Luniya
On May 20, 2016, 6:14 p.m., Juanjo Marron 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/47922/#review135749
---
Ship it!
Ship It!
- Jayush Luniya
On May 26, 2016, 10:59
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46727/#review135657
---
Ship it!
Ship It!
- Jayush Luniya
On May 31, 2016, 12:53
> On May 18, 2016, 11:53 p.m., Jayush Luniya wrote:
> > @Jeff
> > Can you verify that blueprint deployment of the cluster without Livy Server
> > works?
> >
> > Guide on Blueprints
> > https://cwiki.apache.org/confluence/display/AMBARI/Blueprints
>
>
> (Updated May 23, 2016, 2:55 a.m.)
>
>
> Review request for Ambari, Jayush Luniya and Sumit Mohanty.
>
>
> Bugs: AMBARI-16753
> https://issues.apache.org/jira/browse/AMBARI-16753
>
>
> Repository: ambari
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47722/#review134934
---
Ship it!
Ship It!
- Jayush Luniya
On May 25, 2016, 10:38
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47830/#review134932
---
Ship it!
Ship It!
- Jayush Luniya
On May 26, 2016, 5:29 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47845/#review134907
---
Ship it!
Ship It!
- Jayush Luniya
On May 25, 2016, 7:13 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47774/#review134905
---
Ship it!
Ship It!
- Jayush Luniya
On May 24, 2016, 1:27 p.m
201 - 300 of 472 matches
Mail list logo