---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27319/#review58976
---
Ship it!
Ship It!
- Nate Cole
On Oct. 28, 2014, 11:36 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27331/#review58990
---
Ship it!
Ship It!
- Nate Cole
On Oct. 29, 2014, 11:17 a.m
+1 for the proposal.
What will be the process for turning off all the hadoop-centric alerts from the
running Nagios instance? That we remove the service from Ambari still leaves a
dangling service.
Thanks,
Nate
On Oct 29, 2014, at 2:47 PM, Jonathan Hurley jhur...@hortonworks.com wrote:
Hi
see context: are tehse getting set
even if, say, Slider is not installed? Are properties getting set if they
don't belong?
- Nate Cole
On Oct. 29, 2014, 8:48 p.m., Robert Nettleton wrote:
---
This is an automatically generated e-mail
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27372/#review59182
---
Ship it!
Ship It!
- Nate Cole
On Oct. 29, 2014, 11:31 p.m
On Oct. 30, 2014, 7:22 a.m., Nate Cole wrote:
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java,
lines 1017-1019
https://reviews.apache.org/r/27366/diff/1/?file=742190#file742190line1017
Just a general question since I
On Oct. 30, 2014, 7:25 a.m., Nate Cole wrote:
Ship It!
Szilard Nemethy wrote:
Nate,
Will you commit this patch as it is or wait for the implementation for
HDP2.2? Shivani suggested to continue with the work for HDP 2.2 under the
same patch and ticket, but if you
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27394/#review59325
---
Ship it!
Ship It!
- Nate Cole
On Oct. 30, 2014, 4:44 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27422/#review59326
---
Ship it!
Ship It!
- Nate Cole
On Oct. 31, 2014, 6:49 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27435/#review59397
---
Ship it!
Ship It!
- Nate Cole
On Oct. 31, 2014, 2:29 p.m., Tom
[INFO] Finished at: Fri Oct 31 19:26:24 EDT 2014
[INFO] Final Memory: 28M/158M
[INFO]
Thanks,
Nate Cole
]
[INFO] Total time: 49.107s
[INFO] Finished at: Fri Oct 31 19:26:24 EDT 2014
[INFO] Final Memory: 28M/158M
[INFO]
Thanks,
Nate Cole
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27485/#review59521
---
Ship it!
Ship It!
- Nate Cole
On Nov. 2, 2014, 12:01 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27465/#review59537
---
Ship it!
Ship It!
- Nate Cole
On Nov. 1, 2014, 11:22 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27206/#review59570
---
Ship it!
Ship It!
- Nate Cole
On Oct. 30, 2014, 10:09 p.m
12:12:05 EST 2014
[INFO] Final Memory: 29M/213M
[INFO]
Thanks,
Nate Cole
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27530/#review59595
---
On Nov. 3, 2014, 12:32 p.m., Nate Cole wrote
the method, delimited by triple-quotes:
def perform_backup(self):
User to perform the actual backup
- Nate Cole
On Nov. 3, 2014, 4:48 a.m., Szilard Nemethy wrote:
---
This is an automatically generated e-mail
: 29M/248M
[INFO]
Thanks,
Nate Cole
---
On Nov. 4, 2014, 4:55 p.m., Nate Cole wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27590
---
On Nov. 4, 2014, 4:55 p.m., Nate Cole wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27590
is in no-mans land. It's ok to put this kind of detail with
the class doc.
ambari-server/src/main/java/org/apache/ambari/server/state/ServiceComponentHost.java
https://reviews.apache.org/r/27607/#comment101259
Use UpgradeState
- Nate Cole
On Nov. 4, 2014, 11:30 p.m., Alejandro Fernandez wrote
actual timestamp fields, just longs
ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql
https://reviews.apache.org/r/27607/#comment101267
No timestamp fields
- Nate Cole
On Nov. 4, 2014, 11:30 p.m., Alejandro Fernandez wrote
/ClusterVersionEntity.java
https://reviews.apache.org/r/27607/#comment101263
All members should be together at the top. Not member/getter/setter all
together
- Nate Cole
On Nov. 4, 2014, 11:30 p.m., Alejandro Fernandez wrote
.
- Nate Cole
On Nov. 5, 2014, 2:15 p.m., Yurii Shylov wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27635/
---
(Updated Nov
/
---
(Updated Nov. 4, 2014, 11:30 p.m.)
Review request for Ambari, Dmitro Lisnichenko, Jonathan Hurley, Nate Cole,
Sid Wagle, and Yurii Shylov.
Bugs: AMBARI-8148
https://issues.apache.org/jira/browse/AMBARI-8148
Repository: ambari
Description
---
1. Create
/resources/Ambari-DDL-MySQL-CREATE.sql
https://reviews.apache.org/r/27635/#comment101498
Use _ for table name readability: repo_version. We're trying to make
things a bit more readable.
- Nate Cole
On Nov. 6, 2014, 7:59 a.m., Yurii Shylov wrote
https://reviews.apache.org/r/27607/#comment101503
formatting
- Nate Cole
On Nov. 5, 2014, 5:02 p.m., Alejandro Fernandez wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27607
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27635/#review60175
---
Ship it!
Ship It!
- Nate Cole
On Nov. 6, 2014, 11:14 a.m
/ServiceModule.java
https://reviews.apache.org/r/27696/#comment101593
Typically see the java/javax.* imported before the org.* stuff, then com.*
after that.
- Nate Cole
On Nov. 6, 2014, 2:59 p.m., John Speidel wrote
/services/AmbariMetaInfo.java
https://reviews.apache.org/r/27688/#comment101608
Isn't it the case that this could happen?
- Nate Cole
On Nov. 6, 2014, 12:48 p.m., Jonathan Hurley wrote:
---
This is an automatically generated e-mail
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27607/#review60262
---
Ship it!
Ship It!
- Nate Cole
On Nov. 6, 2014, 5:03 p.m
Hi Raveesh,
You can pass data directly to the agent by specifying part of the request info
using the _name of the component itself_. For example, Flume agents are a
many-per-host, and they are individually controlled with this mechanism.
Consider the following command to stop Flume:
PUT
do that
since I don't own it).
- Nate Cole
On Oct. 31, 2014, 10:20 a.m., Szilard Nemethy wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27282
://reviews.apache.org/r/27513/#comment101679
Typically wouldn't use a main like this. Usually you create an instance of
the class and call the methods on it:
backupRestore = BackupRestore(...)
backupRestore.backup()
etc.
- Nate Cole
On Nov. 7, 2014, 3:36 a.m., Szilard Nemethy wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27513/#review60315
---
Ship it!
Ship It!
- Nate Cole
On Nov. 7, 2014, 3:36 a.m
/
---
(Updated Nov. 6, 2014, 2:59 p.m.)
Review request for Ambari, Nate Cole and Tom Beerbower.
Bugs: AMBARI-7175
https://issues.apache.org/jira/browse/AMBARI-7175
Repository: ambari
Description
---
Adds Explicit Stack Service
]
[INFO] Total time: 30:03.553s
[INFO] Finished at: Fri Nov 07 10:55:25 EST 2014
[INFO] Final Memory: 29M/252M
[INFO]
Thanks,
Nate Cole
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27726/#review60347
---
Ship it!
Ship It!
- Nate Cole
On Nov. 7, 2014, 12:43 p.m., Tom
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27696/#review60350
---
Ship it!
Ship It!
- Nate Cole
On Nov. 6, 2014, 2:59 p.m., John
]
Thanks,
Nate Cole
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27740/#review60369
---
Ship it!
Ship It!
- Nate Cole
On Nov. 7, 2014, 3:04 p.m
] Finished at: Sat Nov 08 17:02:15 EST 2014
[INFO] Final Memory: 28M/166M
[INFO]
Thanks,
Nate Cole
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27728/#review60580
---
Ship it!
Ship It!
- Nate Cole
On Nov. 7, 2014, 9:53 a.m
]
Thanks,
Nate Cole
://reviews.apache.org/r/27790/#comment101948
Why change to public here, I didn't see any cases outside the classes of
using it
- Nate Cole
On Nov. 9, 2014, 9:14 a.m., Jonathan Hurley wrote:
---
This is an automatically generated e-mail
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27396/#review60601
---
Ship it!
Ship It!
- Nate Cole
On Nov. 6, 2014, 9:11 p.m., Cabir
On Nov. 10, 2014, 9:51 a.m., Nate Cole wrote:
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertResourceProvider.java,
lines 50-63
https://reviews.apache.org/r/27790/diff/1/?file=755705#file755705line50
Why change to public here, I didn't see any cases
. Can this be in metainfo.xml
somewhere?
- Nate Cole
On Nov. 10, 2014, 4:04 a.m., Cabir Zounaidou wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27802
- Nate Cole
On Nov. 10, 2014, 4:04 a.m., Cabir Zounaidou wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27802/
---
(Updated Nov
/CrudDAO.java
https://reviews.apache.org/r/27819/#comment102006
Generics typically don't get a huge name, so maybe use E/P, or E/K or
something
- Nate Cole
On Nov. 10, 2014, 1:34 p.m., Yurii Shylov wrote:
---
This is an automatically
On Nov. 10, 2014, 11:25 a.m., Nate Cole wrote:
ambari-server/src/main/resources/log_handler_config.json, lines 2-13
https://reviews.apache.org/r/27802/diff/2/?file=756646#file756646line2
Will be hard to add logging for new services. Can this be in
metainfo.xml somewhere
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27870/#review60806
---
Ship it!
Ship It!
- Nate Cole
On Nov. 11, 2014, 11:08 a.m
/dao/ClusterVersionDAO.java
https://reviews.apache.org/r/27854/#comment102220
Use DaoUtils
- Nate Cole
On Nov. 10, 2014, 10:19 p.m., Alejandro Fernandez wrote:
---
This is an automatically generated e-mail. To reply, visit:
https
/ServerActionExecutor.java
https://reviews.apache.org/r/27700/#comment102237
* its (not it is)
- Nate Cole
On Nov. 10, 2014, 9:05 a.m., Robert Levas wrote:
---
This is an automatically generated e-mail. To reply, visit:
https
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27854/#review60844
---
Ship it!
Ship It!
- Nate Cole
On Nov. 11, 2014, 3:16 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27897/#review61029
---
Ship it!
Ship It!
- Nate Cole
On Nov. 11, 2014, 8:15 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27819/#review61030
---
Ship it!
Ship It!
- Nate Cole
On Nov. 12, 2014, 6:10 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27929/#review61081
---
Ship it!
This change fills me with happiness :)
- Nate Cole
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27998/#review61277
---
Ship it!
Ship It!
- Nate Cole
On Nov. 13, 2014, 12:42 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27999/#review61295
---
Ship it!
Ship It!
- Nate Cole
On Nov. 13, 2014, 12:55 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27998/#review61452
---
Ship it!
Ship It!
- Nate Cole
On Nov. 14, 2014, 11:35 a.m
]
[INFO] Total time: 13:39.225s
[INFO] Finished at: Fri Nov 14 17:16:30 EST 2014
[INFO] Final Memory: 27M/226M
[INFO]
Thanks,
Nate Cole
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28105/#review61688
---
Ship it!
Ship It!
- Nate Cole
On Nov. 16, 2014, 8:54 a.m
:21 EST 2014
[INFO] Final Memory: 29M/629M
[INFO]
Thanks,
Nate Cole
]
Thanks,
Nate Cole
---
On Nov. 17, 2014, 6:49 a.m., Nate Cole wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28112
On Nov. 17, 2014, 8:37 a.m., Nate Cole wrote:
ambari-server/src/main/resources/custom_actions/scripts/install_packages.py,
lines 81-85
https://reviews.apache.org/r/27919/diff/3/?file=763819#file763819line81
A repo file may contain many, does this need a loop? Do
On Nov. 17, 2014, 8:37 a.m., Nate Cole wrote:
ambari-server/src/main/resources/custom_actions/scripts/install_packages.py,
lines 81-85
https://reviews.apache.org/r/27919/diff/3/?file=763819#file763819line81
A repo file may contain many, does this need a loop? Do
/#comment103625
It can't be inlined because it needs to be serialized into JSON, and for
that I need the field. Actually, this is a bug in PERCENT that it _shouldn't_
be in lined.
- Nate Cole
On Nov. 17, 2014, 6:49 a.m., Nate Cole wrote
]
Thanks,
Nate Cole
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28112/#review61767
---
On Nov. 17, 2014, 1:56 p.m., Nate Cole wrote:
---
This is an automatically generated e-mail
vs cluster
should look the same, no?
- Nate Cole
On Nov. 17, 2014, 1:28 p.m., Yurii Shylov wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28124
On Nov. 17, 2014, 3:49 p.m., Nate Cole wrote:
Is there a need for two different types? A repo_version on a host vs
cluster should look the same, no?
Alejandro Fernandez wrote:
Technically, there is a difference because the Cluster is required to
have exactly one stack version
/28124/#comment103756
No need. Each provider can live on its own.
- Nate Cole
On Nov. 17, 2014, 1:28 p.m., Yurii Shylov wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28124
://reviews.apache.org/r/28124/
---
(Updated Nov. 17, 2014, 1:28 p.m.)
Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and Nate
Cole.
Bugs: AMBARI-8353
https://issues.apache.org/jira/browse/AMBARI-8353
-DROP.sql
https://reviews.apache.org/r/28159/#comment103852
Since there's a FK, I'm not sure if this needs to be removed before
alert_target. I don't understand SQLServer enough to know
- Nate Cole
On Nov. 18, 2014, 12:34 a.m., Jonathan Hurley wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28159/#review61924
---
Ship it!
Ship It!
- Nate Cole
On Nov. 18, 2014, 9:16 a.m
://reviews.apache.org/r/27284/#comment103847
Why x? There should be no runnables in this directory?
- Nate Cole
On Nov. 13, 2014, 5:39 a.m., Szilard Nemethy wrote:
---
This is an automatically generated e-mail. To reply, visit:
https
://reviews.apache.org/r/28000/#comment103860
change to command_prefix. And is the prefix needed when running as root
(no bash -c)?
- Nate Cole
On Nov. 18, 2014, 10:20 a.m., Andrew Onischuk wrote:
---
This is an automatically generated e-mail
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28000/#review61930
---
Ship it!
Ship It!
- Nate Cole
On Nov. 18, 2014, 10:20 a.m
On Nov. 17, 2014, 3:49 p.m., Nate Cole wrote:
Is there a need for two different types? A repo_version on a host vs
cluster should look the same, no?
Alejandro Fernandez wrote:
Technically, there is a difference because the Cluster is required to
have exactly one stack version
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28194/#review62153
---
Ship it!
Ship It!
- Nate Cole
On Nov. 19, 2014, 8:25 a.m
injected here, so no need for this (a
resource provider is explicitly constructed)
- Nate Cole
On Nov. 19, 2014, 12:41 p.m., Yurii Shylov wrote:
---
This is an automatically generated e-mail. To reply, visit:
https
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27284/#review62331
---
Ship it!
Ship It!
- Nate Cole
On Nov. 19, 2014, 9:30 a.m
]
[INFO] Total time: 20:30.942s
[INFO] Finished at: Thu Nov 20 01:05:35 EST 2014
[INFO] Final Memory: 32M/611M
[INFO]
Thanks,
Nate Cole
results pending
Thanks,
Nate Cole
2014
[INFO] Final Memory: 30M/666M
[INFO]
Thanks,
Nate Cole
/notifications/dispatchers/EmailDispatcher.java
https://reviews.apache.org/r/28291/#comment104546
Nit: does this still work even if there's no markup in the body? If
someone _wanted_ plain text?
- Nate Cole
On Nov. 20, 2014, 2:16 p.m., Jonathan Hurley wrote
.
- Nate Cole
On Nov. 20, 2014, 9:43 p.m., Nate Cole wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28311/
---
(Updated Nov. 20
://reviews.apache.org/r/28311/#review62541
---
On Nov. 20, 2014, 9:43 p.m., Nate Cole wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28311
it?
Nate Cole wrote:
Let's discuss. Not sure why distribute/install requires a restart.
Firing a single custom action that is not a restart is pretty straightforward.
Dmitro Lisnichenko wrote:
Since different hosts with different service components installed have
different package
/ambari/server/api/resources/StageResourceDefinition.java
https://reviews.apache.org/r/28336/#comment104736
Should be stage?
- Nate Cole
On Nov. 21, 2014, 12:26 p.m., Tom Beerbower wrote:
---
This is an automatically generated e-mail
+1 for 1.7.0
On Nov 19, 2014 5:36 PM, Alejandro Fernandez alejan...@apache.org wrote:
Hi all,
The time has come to vote on releasing Ambari 1.7.0.
In the past couple of weeks, the Ambari 1.7.0 release branch has received a
lot of bug fixes and finally reached 0 open Jiras; you should all be
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28336/#review62730
---
Ship it!
Ship It!
- Nate Cole
On Nov. 22, 2014, 8:48 a.m., Tom
]
Thanks,
Nate Cole
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28336/#review62736
---
Ship it!
Ship It!
- Nate Cole
On Nov. 22, 2014, 9:18 a.m., Tom
/HostInfo_linux.py
https://reviews.apache.org/r/28394/#comment105009
Probably also modify HostInfo_win.py
- Nate Cole
On Nov. 24, 2014, 1:33 p.m., Jonathan Hurley wrote:
---
This is an automatically generated e-mail. To reply, visit:
https
across all hosts?
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
https://reviews.apache.org/r/28405/#comment105169
No sysout here.
- Nate Cole
On Nov. 24, 2014, 3:50 p.m., Dmitro Lisnichenko wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28453/#review63052
---
Ship it!
Ship It!
- Nate Cole
On Nov. 25, 2014, 4:53 p.m
1 - 100 of 2298 matches
Mail list logo
