, right after the cache is
filled up.
- Dmytro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47595/#review134093
---------
-site.xml
a7fd9c8
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/service_check.py
8f369f7
Diff: https://reviews.apache.org/r/47595/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
/apache/ambari/server/upgrade/UpgradeCatalog240.java
729b577
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog240Test.java
20fa50f
Diff: https://reviews.apache.org/r/47651/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47531/#review134164
---
Ship it!
Ship It!
- Dmytro Sen
On Май 20, 2016, 3:27 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47701/#review134322
---
Ship it!
Ship It!
- Dmytro Sen
On Май 23, 2016, 5:59 д.п
-site.xml
e5758bf
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/service_check.py
8f369f7
Diff: https://reviews.apache.org/r/47595/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
/sink/timeline/cache/TimelineMetricsCacheTest.java
18d973c
Diff: https://reviews.apache.org/r/47730/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47734/#review134389
---
Ship it!
Ship It!
- Dmytro Sen
On Май 23, 2016, 5:10 п.п
enerated e-mail. To reply, visit:
https://reviews.apache.org/r/47730/#review134370
-------
On Май 23, 2016, 3:48 п.п., Dmytro Sen wrote:
>
> ---
> This is an automati
Diff: https://reviews.apache.org/r/47770/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
faced a
> > problem here before. Can we make this shaded ?
>
> Dmytro Sen wrote:
> Do you mean use https://maven.apache.org/plugins/maven-shade-plugin ? I
> would propose to use same 14.0.1 for ambari-metrics-common
>
> Sid Wagle wrote:
> Yes but the parent had
Get_num_ops ?
- Dmytro Sen
On Май 25, 2016, 10:04 п.п., Aravindan Vijayan wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://rev
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47879/#review134989
---
Ship it!
Ship It!
- Dmytro Sen
On Май 26, 2016, 10:27 д.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47856/#review135195
---
Ship it!
Ship It!
- Dmytro Sen
On Май 26, 2016, 7:16 п.п
://reviews.apache.org/r/47953/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
SUCCESS [02:47 min]
[INFO]
[INFO] BUILD SUCCESS
Thanks,
Dmytro Sen
/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
://reviews.apache.org/r/47730/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
-server/pom.xml 20d3fab
Diff: https://reviews.apache.org/r/47730/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
---
Unit tests passed. + Manual testing
Thanks,
Dmytro Sen
]
[INFO] BUILD SUCCESS
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48087/#review135772
---
Ship it!
Ship It!
- Dmytro Sen
On Май 31, 2016, 7:25 п.п
fd2
Diff: https://reviews.apache.org/r/48121/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
r/internal/BlueprintConfigurationProcessorTest.java
ec04fd2
Diff: https://reviews.apache.org/r/48121/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48139/#review135828
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 1, 2016, 6:06 п.п
> On Май 26, 2016, 3:11 п.п., Dmytro Sen wrote:
> > Ship It!
>
> Qin Liu wrote:
> Thank you so much Dmytro!
>
> Qin Liu wrote:
> Hi Sid,
> Could you please review the patch at your earliest convenience? Thank you
> in advance!
>
> Qin Liu wr
ation/yarn-site.xml
9777ee4
ambari-server/src/main/resources/stacks/HDPWIN/2.2/services/YARN/configuration/yarn-site.xml
c8a588c
Diff: https://reviews.apache.org/r/48121/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
#review135962
-----------
On Июнь 3, 2016, 10:43 д.п., Dmytro Sen wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48121
-server/pom.xml 20d3fab
Diff: https://reviews.apache.org/r/47730/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
2fbb1e1
ambari-server/pom.xml 20d3fab
Diff: https://reviews.apache.org/r/47730/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
ForTesting.class
- Dmytro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47730/#review136064
---
On Июнь 3, 2016, 1:54 п.п., Dmytro Se
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48283/#review136274
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 6, 2016, 3:31 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48282/#review136282
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 6, 2016, 3:18 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48262/#review136288
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 6, 2016, 6:26 д.п
> On Июнь 6, 2016, 4:32 п.п., Dmytro Sen wrote:
> > Ship It!
Committed to trunk and branch-2.4.
Please close this review board
- Dmytro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.or
st.java
8b1a9a6
Diff: https://reviews.apache.org/r/48321/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
ari/server/controller/internal/BlueprintConfigurationProcessorTest.java
8b1a9a6
Diff: https://reviews.apache.org/r/48321/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
d AMS HA
- Dmytro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48321/#review136457
---
On Июнь 7, 2016, 8:12 д.п., Dmytro Sen wrote:
>
> -
ari/server/controller/internal/BlueprintConfigurationProcessorTest.java
8b1a9a6
Diff: https://reviews.apache.org/r/48321/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
o.xml
c6ee13d
Diff: https://reviews.apache.org/r/48402/diff/
Testing
---
Unit tests passed.
Thanks,
Dmytro Sen
ternal/BlueprintConfigurationProcessor.java
de70a2c
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/Stack.java
ad8d4f9
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
9ec0a09
Diff: https://reviews.apache.org/r/48494/diff/
Testing
---
Unit tests and manual tests passed
Thanks,
Dmytro Sen
the server is least loaded.
- Dmytro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48494/#review136823
-------
On Июнь 9, 2016, 3:23 п.п., Dmytro Sen wrote:
>
> ---
s is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48494/#review136832
-----------
On Июнь 9, 2016, 3:23 п.п., Dmytro Sen wrote:
>
> ---
> This
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48508/#review136973
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 10, 2016, полночь
> >
> > Please consider cyclomatic complexity in this method.
>
> Dmytro Sen wrote:
> What is wrong with it ? The loop body is executed only once for every
> property.
> Moreover, this method is executed only once during ambari server
> lifecycle, w
/services/YARN/metainfo.xml
2c3afb0
Diff: https://reviews.apache.org/r/48555/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48641/#review137305
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 13, 2016, 2:31 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48654/#review137478
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 14, 2016, 10:41 д.п
/server/controller/internal/TrimmingStrategy.java
PRE-CREATION
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
cda8fb8
Diff: https://reviews.apache.org/r/48494/diff/
Testing
---
Unit tests and manual tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48706/#review137693
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 15, 2016, 8:43 д.п
/server/controller/internal/TrimmingStrategy.java
PRE-CREATION
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
cda8fb8
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackTest.java
e70af3e
Diff: https://reviews.apache.org/r/48494/diff/
Testing
---
Unit tests and manual tests passed
Thanks,
Dmytro Sen
Thanks,
Dmytro Sen
/server/controller/internal/TrimmingStrategy.java
PRE-CREATION
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
cda8fb8
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StackTest.java
e70af3e
Diff: https://reviews.apache.org/r/48494/diff/
Testing
---
Unit tests and manual tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48729/#review137985
-------
On Июнь 15, 2016, 1:35 п.п., Dmytro Sen wrote:
>
> ---
g/r/48729/#review137985
---
On Июнь 15, 2016, 1:35 п.п., Dmytro Sen wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews
e some way to check if this property was set in the
> > Blueprint, or is relying on the stack defaults.
> >
> > I realize that this change is not really a part of this current patch,
> > but this looks like something that needs some investigation, and
e some way to check if this property was set in the
> > Blueprint, or is relying on the stack defaults.
> >
> > I realize that this change is not really a part of this current patch,
> > but this looks like something that needs some investigation, and
e some way to check if this property was set in the
> > Blueprint, or is relying on the stack defaults.
> >
> > I realize that this change is not really a part of this current patch,
> > but this looks like something that needs some investigation, and
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48852/#review138220
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 17, 2016, 1:47 п.п
/
Diffs
-
ambari-metrics/ambari-metrics-timelineservice/pom.xml 2fefede
Diff: https://reviews.apache.org/r/48858/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48985/#review138803
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 21, 2016, 1:41 д.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49018/#review138810
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 21, 2016, 9:55 д.п
> On Июнь 21, 2016, 8:44 д.п., Dmytro Sen wrote:
> > Ship It!
committed to trunk and branch-2.4
- Dmytro
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48985/#rev
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47730/#review136067
---
On Июнь 3, 2016, 3:15 п.п., Dmytro Sen wrote:
>
> -
passed
Thanks,
Dmytro Sen
-
[INFO] BUILD SUCCESS
[INFO]
[INFO] Total time: 1:08.949s
[INFO] Finished at: Tue Jun 21 18:52:10 EEST 2016
[INFO] Final Memory: 70M/948M
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49032/#review138866
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 21, 2016, 4:28 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49080/#review139052
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 22, 2016, 11:12 д.п
/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49153/#review139252
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 23, 2016, 1:56 п.п
.xml
985e7f4
ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py
2bdaa48
ambari-server/src/main/resources/stacks/HDP/2.1.GlusterFS/services/FALCON/configuration/falcon-client.properties.xml
PRE-CREATION
ambari-server/src/main/resources/stacks/HDP/2.1.GlusterFS/services/FALCON/metainfo.xml
2e444b1
ambari-server/src/main/resources/stacks/HDP/2.1.GlusterFS/services/FALCON/package/scripts/params.py
41ecafa
Diff: https://reviews.apache.org/r/49199/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49205/#review139380
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 24, 2016, 4:32 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49263/#review139562
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 27, 2016, 12:42 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49372/#review139989
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 29, 2016, 2:24 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49454/#review140174
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 30, 2016, 3:23 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49504/#review140331
---
Ship it!
Ship It!
- Dmytro Sen
On Июль 1, 2016, 10:51 д.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48852/#review140347
---
Ship it!
Ship It!
- Dmytro Sen
On Июнь 17, 2016, 2:54 п.п
---
Unit tests passed
Thanks,
Dmytro Sen
://reviews.apache.org/r/49513/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
----------
On Июль 1, 2016, 3:26 п.п., Dmytro Sen wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49513/
>
://reviews.apache.org/r/49514/diff/
Testing
---
Unit testst passed
Thanks,
Dmytro Sen
catch, however, why not look for "._" and replace that only if
> > needed.
There is no change here. It's a copy of previous StormTimelineMetricsSink.java
- Dmytro
---
This is an automatically gen
--
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49514/#review140459
-----------
On Июль 1, 2016, 3:32 п.п., Dmytro Sen wrote:
>
>
---
Unit testst passed
Thanks,
Dmytro Sen
://reviews.apache.org/r/49589/diff/
Testing
---
Unit tests passed + manual testing
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49590/#review140653
---
Ship it!
Ship It!
- Dmytro Sen
On Июль 4, 2016, 1:11 п.п
urces/stacks/HDP/2.1/role_command_order.json
3adf5e1
Diff: https://reviews.apache.org/r/49594/diff/
Testing
---
Manual unit tests
Thanks,
Dmytro Sen
ply, visit:
https://reviews.apache.org/r/49594/#review140664
-----------
On Июль 4, 2016, 2:52 п.п., Dmytro Sen wrote:
>
> ---
> This is an automatically ge
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49702/#review140979
---
Ship it!
Ship It!
- Dmytro Sen
On Июль 6, 2016, 11:37 д.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49590/#review140993
---
Ship it!
Ship It!
- Dmytro Sen
On Июль 6, 2016, 1:43 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49749/#review141152
---
Ship it!
Ship It!
- Dmytro Sen
On Июль 7, 2016, 12:08 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49762/#review141173
---
Ship it!
Ship It!
- Dmytro Sen
On Июль 7, 2016, 3:43 п.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49801/#review141267
---
Ship it!
Ship It!
- Dmytro Sen
On Июль 8, 2016, 10:30 д.п
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49803/#review141268
---
Ship it!
Ship It!
- Dmytro Sen
On Июль 8, 2016, 10:32 д.п
error, upgrade failed
com.google.inject.ProvisionException: Guice provision errors:
Diffs
-
ambari-server/src/main/java/org/apache/ambari/server/orm/DBAccessorImpl.java
b10d32a
Diff: https://reviews.apache.org/r/49805/diff/
Testing
---
Unit tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49808/#review141701
---
Ship it!
Ship It!
- Dmytro Sen
On Июль 11, 2016, 3:29 п.п
Thanks,
Dmytro Sen
ain/resources/common-services/STORM/0.9.1/package/scripts/storm.py
> bce71ca
>
> ambari-server/src/main/resources/common-services/STORM/0.9.1/package/scripts/ui_server.py
> ef2c536
>
> Diff: https://reviews.apache.org/r/49902/diff/
>
>
> Testing
> ---
>
> Unit tests passed
>
>
> Thanks,
>
> Dmytro Sen
>
>
---
Unit tests passed
Thanks,
Dmytro Sen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49982/#review142037
---
Ship it!
Ship It!
- Dmytro Sen
On Июль 13, 2016, 8:55 д.п
101 - 200 of 529 matches
Mail list logo