---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61767/#review183292
---
Ship it!
Ship It!
- Sumit Mohanty
On Aug. 19, 2017, 3:18
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61713/#review183127
---
Ship it!
Ship It!
- Sumit Mohanty
On Aug. 17, 2017, 1:15
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61623/#review182927
---
Ship it!
Ship It!
- Sumit Mohanty
On Aug. 15, 2017, 1:43
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61587/#review182646
---
Ship it!
Ship It!
- Sumit Mohanty
On Aug. 10, 2017, 11:36
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61547/#review182596
---
Ship it!
Ship It!
- Sumit Mohanty
On Aug. 10, 2017, 3:05
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61422/#review182186
---
Ship it!
Ship It!
- Sumit Mohanty
On Aug. 4, 2017, 4:06 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61377/#review181988
---
Ship it!
Ship It!
- Sumit Mohanty
On Aug. 2, 2017, 1:21 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61228/#review181725
---
Ship it!
Ship It!
- Sumit Mohanty
On July 28, 2017, 9:12
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61203/#review181701
---
Ship it!
Ship It!
- Sumit Mohanty
On July 28, 2017, 4:50
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61217/#review181668
---
Ship it!
Ship It!
- Sumit Mohanty
On July 28, 2017, 2:13
o
that both instances do not retry at the same time?
- Sumit Mohanty
On July 28, 2017, 4:50 a.m., Aravindan Vijayan 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/61083/#review181603
---
Ship it!
Ship It!
- Sumit Mohanty
On July 27, 2017, 6:49
/upgrades/config-upgrade.xml
Lines 268 (patched)
<https://reviews.apache.org/r/61178/#comment257158>
Can it be deleted unconditionally or only if the value is exactly the hdfs
location? I suspect if the cluster uses some other DFS.
- Sumit Mohanty
On July 27, 2017, 2:28 p.m., Attila Dor
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61140/#review181564
---
Ship it!
Ship It!
- Sumit Mohanty
On July 26, 2017, 3:42
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61168/#review181527
---
Ship it!
Ship It!
- Sumit Mohanty
On July 27, 2017, 2:04
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61163/#review181493
---
Ship it!
Ship It!
- Sumit Mohanty
On July 26, 2017, 9:48
2 (patched)
<https://reviews.apache.org/r/61107/#comment256952>
Do we need this change during the EU?
- Sumit Mohanty
On July 25, 2017, 11:29 p.m., Vitalyi Brodetskyi wrote:
>
> ---
> This is an automatically generated
/package/scripts/metadata.py
Lines 210 (patched)
<https://reviews.apache.org/r/61083/#comment256912>
Multiple copies of core-site is fine isn't it. Assuming Atlas is now marked
for restart when core-site is changed.
- Sumit Mohanty
On July 24, 2017, 2:51 p.m., Vishal Su
/UpgradeCatalog252.java
Lines 310 (patched)
<https://reviews.apache.org/r/61050/#comment256583>
Should this have been done as part of the upgrade pack instead of Ambari
upgrade?
- Sumit Mohanty
On July 21, 2017, 8:18 p.m., Di Li
/package/scripts/oozie.py
Lines 287 (patched)
<https://reviews.apache.org/r/61012/#comment256513>
Earlier the code would log/fail if path did not exist. But now it will
skip. So if there is no successful copy should we log that nothing got copied?
- Sumit Mohanty
On July 20, 2017,
<https://reviews.apache.org/r/60921/#comment255977>
With the java change, will this error path be hit?
- Sumit Mohanty
On July 17, 2017, 8:14 p.m., Vitalyi Brodetskyi wrote:
>
> ---
> This is an automatically generated e-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60919/#review180747
---
Ship it!
Ship It!
- Sumit Mohanty
On July 17, 2017, 9:31
/upgrades/config-upgrade.xml
Lines 177 (patched)
<https://reviews.apache.org/r/60919/#comment255949>
Will this set the value only if the property does not exist so that user
supplied value is not over-written?
- Sumit Mohanty
On July 17, 2017, 7:36 p.m., Alejandro Fernandez
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60878/#review180584
---
Ship it!
Ship It!
- Sumit Mohanty
On July 14, 2017, 8:09
> 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.
>
> Sumit Mohanty wrote:
> +1. It will good to not have Ambari upgrade add any config or an absolute
>
--
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60741/
> ---
>
> (Updated July 10, 2017, 12:30 p.m.)
>
>
> Review request for Ambari, Alejan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60560/#review179483
---
Ship it!
Ship It!
- Sumit Mohanty
On July 3, 2017, 5:35 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60656/#review179658
---
Ship it!
Ship It!
- Sumit Mohanty
On July 5, 2017, 5:20 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60602/#review179514
---
Ship it!
Ship It!
- Sumit Mohanty
On July 3, 2017, 2:42 p.m
/configuration/atlas-env.xml
Lines 185 (patched)
<https://reviews.apache.org/r/60560/#comment254219>
Nit: Spelling
- Sumit Mohanty
On June 30, 2017, 8:49 a.m., Vishal Suvagia wrote:
>
> ---
> This is an automatically g
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60571/#review179413
---
Ship it!
Ship It!
- Sumit Mohanty
On June 30, 2017, 8:36
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60408/#review178832
---
Ship it!
Ship It!
- Sumit Mohanty
On June 24, 2017, 12:42
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60388/#review178797
---
Ship it!
Ship It!
- Sumit Mohanty
On June 23, 2017, 1:13
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60365/#review178715
---
Ship it!
Ship It!
- Sumit Mohanty
On June 22, 2017, 11:34
N that config update API got a call with same tag.
- Sumit Mohanty
On June 22, 2017, 11:34 a.m., Vitalyi Brodetskyi 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/60126/#review178324
---
Ship it!
Ship It!
- Sumit Mohanty
On June 19, 2017, 10:38
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59992/#review177855
---
Ship it!
Ship It!
- Sumit Mohanty
On June 14, 2017, 1:31
> On June 13, 2017, 3:18 p.m., Sumit Mohanty wrote:
> > One more thing: If user upgrades from HDP-2.6.x to 2.6.2, do you want the
> > configs to be added? If yes then that upgrade xml change is needed.
>
> Saisai Shao wrote:
> Yes, I want. So do I only need to chang
> On June 13, 2017, 3:18 p.m., Sumit Mohanty wrote:
> > One more thing: If user upgrades from HDP-2.6.x to 2.6.2, do you want the
> > configs to be added? If yes then that upgrade xml change is needed.
>
> Saisai Shao wrote:
> Yes, I want. So do I only need to chang
want the
configs to be added? If yes then that upgrade xml change is needed.
- Sumit Mohanty
On June 13, 2017, 5:44 a.m., Saisai Shao wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://review
new configs in
HDP-2.6 are auto added as they did not exist in older versions. You can confirm
this with Jonathan Hurley or Nate Cole as well. SO only change needed is for
upgrade with the HDP-2.6.x.
- Sumit Mohanty
On June 12, 2017, 8:44 a.m., Saisai Shao wrote
thrift-sparkconf.xml
Lines 38 (patched)
<https://reviews.apache.org/r/59992/#comment251314>
Let this be false. These configs should not be added as part of Ambari
upgrade. (Actually lets set it to false for all the four new additions)
- Sumit Mohanty
On June 12, 2017, 8:44 a.m., Saisa
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59364/#review175377
---
Ship it!
Ship It!
- Sumit Mohanty
On May 18, 2017, 10:54
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59338/#review175251
---
Ship it!
Ship It!
- Sumit Mohanty
On May 17, 2017, 2:18 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59295/#review175191
---
Ship it!
Ship It!
- Sumit Mohanty
On May 15, 2017, 10:25
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59305/#review175062
---
Ship it!
Ship It!
- Sumit Mohanty
On May 16, 2017, 5:58 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59113/#review174399
---
Ship it!
Ship It!
- Sumit Mohanty
On May 9, 2017, 10:33 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58993/#review173907
---
Ship it!
Ship It!
- Sumit Mohanty
On May 4, 2017, 2:29 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58965/#review173739
---
Ship it!
Ship It!
- Sumit Mohanty
On May 3, 2017, 3:12 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58960/#review173734
---
Ship it!
Ship It!
- Sumit Mohanty
On May 3, 2017, 11:45 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58756/#review173523
---
Ship it!
Ship It!
- Sumit Mohanty
On April 27, 2017, 12:21
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58884/#review173478
---
Ship it!
Ship It!
- Sumit Mohanty
On May 1, 2017, 5:30 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58860/#review173403
---
Ship it!
Ship It!
- Sumit Mohanty
On April 28, 2017, 11:16
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58862/#review173395
---
Ship it!
Ship It!
- Sumit Mohanty
On April 28, 2017, 11:13
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58739/#review173048
---
Ship it!
Ship It!
- Sumit Mohanty
On April 26, 2017, 2:12
new definition.
- Sumit Mohanty
On April 21, 2017, 9:41 p.m., Alejandro Fernandez 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/58568/#review172489
---
Ship it!
Ship It!
- Sumit Mohanty
On April 20, 2017, 2:19
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58551/#review172451
---
Ship it!
Ship It!
- Sumit Mohanty
On April 20, 2017, 2:28
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58437/#review171957
---
Ship it!
Ship It!
- Sumit Mohanty
On April 13, 2017, 10:21
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58398/#review171780
---
Ship it!
Ship It!
- Sumit Mohanty
On April 12, 2017, 7:21
> On April 8, 2017, 9:06 p.m., Sumit Mohanty wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostRoleCommandDAO.java
> > Lines 945 (patched)
> > <https://reviews.apache.org/r/58282/diff/1/?file=1686812#file1686812line945>
> >
> >
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58282/#review171409
---
Ship it!
Ship It!
- Sumit Mohanty
On April 8, 2017, 8:50
server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql
Lines 387 (patched)
<https://reviews.apache.org/r/58282/#comment244327>
Needs upgrade catalog changes fro 2.5.1
- Sumit Mohanty
On April 8, 2017, 8:50 p.m., Jonathan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57875/#review170642
---
Ship it!
Ship It!
- Sumit Mohanty
On March 30, 2017, 9:10
/package/scripts/hive_interactive.py
Line 101 (original), 101 (patched)
<https://reviews.apache.org/r/58031/#comment243329>
Nit, space after comma - few occurrences in the code.
- Sumit Mohanty
On March 29, 2017, 5:45 p.m., Madhuvanthi Radhakrishnan
.
- Sumit Mohanty
On March 29, 2017, 8:20 p.m., Nate Cole wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
m., Aravindan Vijayan wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58006/
> ---
>
> (Updated March 28, 2017, 10:03 p.m.)
>
>
> Review request for Ambari
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57909/#review170010
---
Ship it!
Ship It!
- Sumit Mohanty
On March 24, 2017, 12:05
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57906/#review170007
---
Ship it!
Ship It!
- Sumit Mohanty
On March 24, 2017, 8:37
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57785/#review169472
---
Ship it!
Ship It!
- Sumit Mohanty
On March 20, 2017, 5:35
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57684/#review169154
---
Ship it!
Ship It!
- Sumit Mohanty
On March 16, 2017, 11:41
support?
- Sumit Mohanty
On March 16, 2017, 12:21 a.m., Madhuvanthi Radhakrishnan 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/57672/#review169087
---
Ship it!
Ship It!
- Sumit Mohanty
On March 16, 2017, 12:21
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57669/#review169081
---
Ship it!
Ship It!
- Sumit Mohanty
On March 15, 2017, 11:31
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57625/#review169022
---
Ship it!
Ship It!
- Sumit Mohanty
On March 14, 2017, 10:35
. SUCCESS [8.301s]
[INFO] Ambari Agent .. SUCCESS [1:36.871s]
[INFO]
[INFO] BUILD SUCCESS
Thanks,
Sumit Mohanty
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57537/#review168712
---
Ship it!
Ship It!
- Sumit Mohanty
On March 11, 2017, 7:07
properties file - would it
cause trouble when service read it - e.g. when Spark is starting? IOW, do they
need to set something in the env file to be able to read UTF-8.
- Sumit Mohanty
On March 11, 2017, 11:43 a.m., Andrew Onischuk wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57509/#review168644
---
Ship it!
Ship It!
- Sumit Mohanty
On March 10, 2017, 4:34
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57297/#review168361
---
Ship it!
Ship It!
- Sumit Mohanty
On March 3, 2017, 6:59
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57423/#review168304
---
Ship it!
Ship It!
- Sumit Mohanty
On March 8, 2017, 4:04
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57292/#review167998
---
Ship it!
Ship It!
- Sumit Mohanty
On March 6, 2017, 3:56
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57075/#review167988
---
Ship it!
Ship It!
- Sumit Mohanty
On March 1, 2017, 7:35
> On March 3, 2017, 9:56 p.m., Sumit Mohanty wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractProviderModule.java
> > Line 178 (original), 178 (patched)
> > <https://reviews.apache.org/r/57293/diff/1/?file=1655490#file1655490lin
/internal/AbstractProviderModule.java
Line 178 (original), 178 (patched)
<https://reviews.apache.org/r/57293/#comment239839>
Should this be an OR - if hbase.ssl.enabled is true or hbase.http.policy is
true?
- Sumit Mohanty
On March 3, 2017, 3:43 p.m., Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57306/#review167873
---
Ship it!
Ship It!
- Sumit Mohanty
On March 3, 2017, 9:52
for Ambari, Alejandro Fernandez, Jayush Luniya, and Sumit
Mohanty.
By Madhuvanthi Radhakrishnan.
Updated March 2, 2017, 7:48 p.m.
Bugs: AMBARI-20275<https://issues.apache.org/jira/browse/AMBARI-20275>
Repository: ambari
Description
Credential Store should be enabled by default on fresh insta
EU.
Thanks,
Sumit Mohanty
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57258/#review167776
---
Ship it!
Ship It!
- Sumit Mohanty
On March 2, 2017, 10:32
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57256/#review167742
---
Ship it!
Ship It!
- Sumit Mohanty
On March 2, 2017, 9:29
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57253/#review167727
---
Ship it!
Ship It!
- Sumit Mohanty
On March 2, 2017, 7:48
/stack_advisor.py (line
1208)
<https://reviews.apache.org/r/57128/#comment239345>
Is this TODO a bug?
ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py (line
1312)
<https://reviews.apache.org/r/57128/#comment239346>
Is this TODO a bug?
- Sumit Mohanty
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57128/#review167176
---
Ship it!
Ship It!
- Sumit Mohanty
On Feb. 28, 2017, 11:07
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57116/#review166981
---
Ship it!
Ship It!
- Sumit Mohanty
On Feb. 28, 2017, 1:18
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57113/#review166960
---
Ship it!
Ship It!
- Sumit Mohanty
On Feb. 27, 2017, 11:06
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57096/#review166915
---
Ship it!
Ship It!
- Sumit Mohanty
On Feb. 27, 2017, 7:10
/package/scripts/kafka_broker.py
(line 25)
<https://reviews.apache.org/r/57096/#comment238994>
Can we remove the import too?
- Sumit Mohanty
On Feb. 27, 2017, 6:34 p.m., Madhuvanthi Radhakrishnan wrote:
>
> ---
> This is a
> On Feb. 26, 2017, 7:43 p.m., Sumit Mohanty wrote:
> > ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2.5/package/scripts/master.py,
> > line 168
> > <https://reviews.apache.org/r/57075/diff/1/?file=1648119#file1648119line168>
> >
> >
r=params.zeppelin_user,
group=params.zeppelin_group,
mode=0644)
Should zeppelin be restarted when these configs change? In that case, can
you add hive-site and hbase-site to config dependencies.
How about HSI or Hive2Server2? Does zeppelin nee
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56974/#review166745
-------
On Feb. 25, 2017, 1:29 a.m., Sumit Mohanty wrote:
>
> ---
&g
1 - 100 of 649 matches
Mail list logo