---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48234/#review136242
---
Ship it!
Ship It!
- Di Li
On June 3, 2016, 10:47 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48702/#review137721
---
Ship it!
Ship It!
- Di Li
On June 14, 2016, 11:12 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47746/#review136501
---
Ship it!
Ship It!
- Di Li
On June 7, 2016, 5:23 p.m
properties are changed when I enabled CPU isolation on the
Yarn config tab.
Thanks,
Di Li
successfully during the EU.
Thanks,
Di Li
---
patch a trunk cluster (Oozie use is non-default myoozie, group is non-default
myhadoop) with the change, run express upgrade, verify that with the change,
oozie restarted successfully during the EU.
Thanks,
Di Li
or message also shows the config_id
Thanks,
Di Li
> On June 16, 2016, 6:34 p.m., Di Li wrote:
> > ambari-web/app/controllers/main/admin/highAvailability/nameNode/step9_controller.js,
> > line 146
> > <https://reviews.apache.org/r/48734/diff/1/?file=1420112#file1420112line146>
> >
> > I am under the i
. As a safeguard against offline active NN, the check should
eventually timeout to unblock the rest of the start sequence.
- Di Li
On June 15, 2016, 4:41 p.m., Victor Galgo wrote:
>
> ---
> This is an automatically generated e-mail.
previous comment to Victor about my
investigation and why I think the fix should happen at the NameNode Python code
level? Let me know if it's a reasonable statement...
- Di Li
On June 15, 2016, 4:41 p.m., Victor Galgo wrote
e some logic to check if NameNode is out of the
safemode? On a cluster with terabytes of data in HDFS, it may take NN quite
some time (a few minutes, depending on the cluster's performenace) to exit the
safemode.
- Di Li
On June 15, 2016, 4:41 p.m., Victor Galgo
rg/r/48855/diff/
Testing
---
1. patch a trunk cluster with the code change, ran express upgrades, express
downgrades, rolling upgrades and rolling downgrades. Check upgrade and
downgrade history displayed on the UI
2. new unit tests
Thanks,
Di Li
submitted?
- Di Li
On May 18, 2016, 7:05 p.m., Anita Jebaraj wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
Diff: https://reviews.apache.org/r/49267/diff/
Testing
---
Build ambari rpms with the changes and run install, verify on the host
registration step, the messages displayed are built at the view level.
Thanks,
Di Li
just as well as the one I proposed.
Thanks. Di
- Di
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49267/#review139580
---
On June
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44972/#review124176
---
Ship it!
Ship It!
- Di Li
On March 17, 2016, 6:11 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44457/#review123464
---
Ship it!
Ship It!
- Di Li
On March 11, 2016, 8:08 p.m
> On March 14, 2016, 5:47 p.m., Di Li wrote:
> > Ship It!
>
> Keta Patel wrote:
> Thank you Di!
can you reattach the latest build via the "Update Diff" option? Your
AMBARI-15310_Mar10.patch is not up to downl
://reviews.apache.org/r/45054/diff/
Testing
---
unit test
patch a trunk cluster with code change, verify the host bulk command list shown
for DataNode, HBase Region server, storm supervisor, etc.
Thanks,
Di Li
server, storm supervisor, etc.
Thanks,
Di Li
h.
- Di
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45054/#review124303
---
On March 29, 2016, 5:03 p.m., Di Li wrote:
>
> ---
> This is an automaticall
manager stops nodemanagers across the cluster.
Thanks,
Di Li
it should go into?
Thanks.
- Di Li
On March 18, 2016, 6:32 p.m., Amruta Borkar wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
://reviews.apache.org/r/45744/diff/
Testing
---
patch a cluster with latest metainfo.xml change, restart cluster and verify now
the bulk command for stop node manager stops nodemanagers across the cluster.
Thanks,
Di Li
---
unit test
patch a trunk cluster with code change, verify the host bulk command list shown
for DataNode, HBase Region server, storm supervisor, etc.
Thanks,
Di Li
a
warning this time, about a same-named property is defined in hbase-site.xml.
This is to test the new behavior introduced via this JIRA.
Thanks,
Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46475/#review130483
---
Ship it!
Ship It!
- Di Li
On April 25, 2016, 4:37 p.m
cluster,
verify /var/lib/ambari-agent/cache/stack and
/var/lib/ambari-agent/cache/common-services are backed up.
Thanks,
Di Li
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46745/#review130833
-----------
On April 27, 2016, 6:19 p.m., Di Li wrote:
>
> ---
&g
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46592/#review130394
---
Ship it!
Ship It!
- Di Li
On April 22, 2016, 11:17 p.m
ux box with only Pyton 2.6 installed,
verifies it has the same content as one generated on a linux box with Python
2.7.
Thanks,
Di Li
d input. Only alphanumerics, underscores, hyphens,
percentage and spaces are allowed."
- Di Li
On April 21, 2016, 10:51 p.m., Keta Patel wrote:
>
> ---
> This is an automatically generated e-mail. To reply, vi
Diff: https://reviews.apache.org/r/47480/diff/
Testing
---
Manually patch a trunk cluster with UI change, verify the label is displayed
correctly.
Thanks,
Di Li
/controllers/clusters/UserAccessListCtrl.js
(line 75)
<https://reviews.apache.org/r/46808/#comment195266>
what happens when permission_label is not None?
- Di Li
On April 29, 2016, 12:41 a.m., Keta Patel
Verify the existing href behavior is still correct.
3. Submit a rest api that does not need to return /requests url, like get stack
details REST API
Veriy the existing href behavior is still correct.
On a non-kerberized trunk cluster, patch the cluster with fix,
Repeat the 3 tests mentioned above.
Thanks,
Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49912/#review142077
---
Ship it!
Ship It!
- Di Li
On July 11, 2016, 6:43 p.m., Keta
-web/app/views/common/quick_view_link_view.js 17d7c04
ambari-web/test/views/common/quick_link_view_test.js f9a52b1
Diff: https://reviews.apache.org/r/50392/diff/
Testing
---
updated unit tests
Thanks,
Di Li
> On July 13, 2016, 4:53 p.m., Di Li wrote:
> > Ship It!
committed, please close the record.
- Di
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49912/#rev
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50539/#review143925
---
Ship it!
Ship It!
- Di Li
On July 27, 2016, 11:06 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50350/#review143539
---
Ship it!
Ship It!
- Di Li
On July 26, 2016, 1:22 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51017/#review145653
---
Ship it!
Ship It!
- Di Li
On Aug. 12, 2016, 3:19 p.m
committed.
Thanks.
Di
- Di Li
On June 29, 2016, 7:31 p.m., Sangeeta Ravindran wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
committed.
Thanks.
Di
- Di Li
On June 27, 2016, 11:20 p.m., Anita Jebaraj wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
python script against a cluster with only ambari server and agents
installed. the python script prints out the same check results as the UI on the
host registration page.
Thanks,
Di Li
ly, visit:
https://reviews.apache.org/r/48855/#review140829
-----------
On July 5, 2016, 7:53 p.m., Di Li wrote:
>
> ---
> This is an automatically generated e-mail. To re
des. Check upgrade and
downgrade history displayed on the UI
2. new unit tests
3. existing unit tests
Thanks,
Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49387/#review141280
---
please add Jonathan Hurley to the review
- Di Li
On June 29
://reviews.apache.org/r/49895/diff/
Testing
---
Run the script against a cluster just Ambari server and Ambari Agent nodes
installed.verify the script can get the correct list of ambari agents.
Thanks,
Di Li
displayed on the UI
2. new unit tests
Thanks,
Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50870/#review145255
---
Ship it!
Ship It!
- Di Li
On Aug. 9, 2016, 5:15 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49287/#review139767
---
Ship it!
Ship It!
- Di Li
On June 27, 2016, 11:20 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49393/#review140043
---
Ship it!
Ship It!
- Di Li
On June 29, 2016, 7:31 p.m
://reviews.apache.org/r/49324/diff/
Testing
---
run python script against a cluster with only ambari server and agents
installed. the python script prints out the same check results as the UI on the
host registration page.
Thanks,
Di Li
i.
- Di
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49324/#review140035
---
On June 29, 2016, 8:49 p.m., Di
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51055/#review145773
---
Ship it!
Ship It!
- Di Li
On Aug. 12, 2016, 11:48 p.m
> On Feb. 3, 2017, 1:28 p.m., Di Li wrote:
> > Ship It!
pushed to trunk as
https://git-wip-us.apache.org/repos/asf?p=ambari.git;a=commit;h=ecfc951b5c8589b9225684c3ad2660cdb4ef4445
- Di
---
This is an automatically generat
---
patched a cluster with the change, run preinstall_check.py script
Thanks,
Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56235/#review164118
---
Ship it!
Ship It!
- Di Li
On Feb. 2, 2017, 8:01 p.m
ade wizard is
opened and paused at a menu step.
existing web UI unit tests.
Thanks,
Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55789/#review162649
---
Please add Nate Cole for the review.
- Di Li
On Jan. 20, 2017
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55789/#review162648
---
Ship it!
Ship It!
- Di Li
On Jan. 20, 2017, 11:49 p.m
d finalize the upgrade.
Thanks,
Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55310/#review162848
---
Ship it!
Ship It!
- Di Li
On Jan. 24, 2017, 7:50 p.m., Shi
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55935/#review163131
---
Ship it!
Ship It!
- Di Li
On Jan. 25, 2017, 8:58 p.m
as my ambari repo id.
Thanks,
Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55993/#review163165
---
Ship it!
Ship It!
- Di Li
On Jan. 26, 2017, 7:44 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55992/#review163164
---
Ship it!
Ship It!
- Di Li
On Jan. 26, 2017, 7:19 p.m
"IN_PROGESS". then be able to click the finalize and finalize the upgrade.
Thanks,
Di Li
> On Feb. 17, 2017, 1:46 p.m., Di Li wrote:
> > Hello Yuan Bo, Please add JIRA ID to the Bugs field and specify the proper
> > branch for the fix.
nm branch, just the JIRA ID
- Di
---
This is an automatically g
A2
UI code needs updates accordingly.
- Di Li
On Feb. 22, 2017, 1:52 a.m., yao lei wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
the proper
branch for the fix.
- Di Li
On Feb. 14, 2017, 3 a.m., Yuanbo Liu wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
> On Feb. 14, 2017, 8 p.m., Di Li wrote:
> > Ship It!
pushed to trunk as
https://git-wip-us.apache.org/repos/asf?p=ambari.git;a=commit;h=9e5c57e3e7ae4a8b3df51efcbddcdb97a1f7b398
Yuanbo, please close this rb reques
> On Feb. 22, 2017, 9:21 p.m., Di Li wrote:
> > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/metainfo.xml,
> > line 109
> > <https://reviews.apache.org/r/55817/diff/3/?file=1641784#file1641784line109>
> >
> > Hello Yao,
> >
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56503/#review164987
---
Ship it!
Ship It!
- Di Li
On Feb. 9, 2017, 6:31 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56627/#review165566
---
Ship it!
Ship It!
- Di Li
On Feb. 14, 2017, 12:37 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56636/#review165564
---
Ship it!
Ship It!
- Di Li
On Feb. 14, 2017, 3 a.m., Yuanbo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55586/#review162664
---
Ship it!
Ship It!
- Di Li
On Jan. 23, 2017, 5:44 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55516/#review161901
---
Ship it!
Ship It!
- Di Li
On Jan. 17, 2017, 5:53 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55523/#review161730
---
Ship it!
Ship It!
- Di Li
On Jan. 13, 2017, 9:43 p.m
/common-services/YARN/3.0.0.3.0/package/scripts/params_linux.py
23a25a0
Diff: https://reviews.apache.org/r/55584/diff/
Testing
---
existing unit tests
build rpms off trunk source code and deploy a cluster.
Thanks,
Di Li
is delivered to trunk ?
- Di Li
On Dec. 22, 2016, 8:59 p.m., Diego Santesteban 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/53213/#review162146
---
Ship it!
Ship It!
- Di Li
On Jan. 12, 2017, 12:33 a.m
/55774/diff/
Testing
---
existing unit tests
build ambari rpms with trunk code, install a trunk cluster.
Thanks,
Di Li
: https://reviews.apache.org/r/55720/diff/
Testing
---
existing unit test
patch trunk code build ambari rpms and install a trunk cluster
Thanks,
Di Li
/VersionUtils.java
d3d8592
ambari-server/src/test/java/org/apache/ambari/server/utils/TestVersionUtils.java
821565e
Diff: https://reviews.apache.org/r/57146/diff/
Testing
---
existing unit tests, new unit tests, trunk cluster install
Thanks,
Di Li
k host operation submitted by the Ambari web UI during Host
registration step.
Current behavior is a hard fail, but I think it's still the correct
behavior nontheless.
- Di Li
On Feb. 25, 2017, 3 a.m., Amr
> On Aug. 12, 2016, 7:53 p.m., Di Li wrote:
> > Ship It!
>
> Sangeeta Ravindran wrote:
> Thanks Di.
Hello Sangeeta, please close this one if it has been committed.
- Di
---
This is an automatically generate
js 59baf07
ambari-web/test/router_test.js e17c9e2
ambari-web/test/views/main/admin/stack_upgrade/version_view_test.js b7df818
Diff: https://reviews.apache.org/r/51604/diff/
Testing
---
existing unit test
added new unit test
Thanks,
Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51578/#review148187
---
Ship it!
Ship It!
- Di Li
On Sept. 7, 2016, 4:57 p.m., Keta
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51761/#review148347
---
Ship it!
Ship It!
- Di Li
On Sept. 9, 2016, 4:59 p.m., Tim
/DependencyConditionInfo.java
(line 35)
<https://reviews.apache.org/r/51815/#comment216449>
should configType, property, value be mapped to xml elements explicitly ?
- Di Li
On Sept. 13, 2016, 9:28 p.m., Amruta Borkar
kicked in). add the properties with a non-default value, verify I can't
add repos.
Thanks,
Di Li
gative test, what if a user manually added the property to
hdfs-site.xml ?
- Di Li
On Sept. 12, 2016, 5:45 p.m., Amruta Borkar wrote:
>
> ---
> This is an automaticall
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51815/#review148722
---
Please add Shantanu as a reviewer
- Di Li
On Sept. 12, 2016, 5
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52396/#review150875
---
Ship it!
Ship It!
- Di Li
On Sept. 29, 2016, 4:40 p.m
/52554/diff/
Testing
---
existing unit test
patch a trunk cluster, install the cluster, then run HostCleanup.py while
feeding it with two .ini file for basic host clean up.
Thanks,
Di Li
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52430/#review151367
---
Ship it!
Ship It!
- Di Li
On Oct. 3, 2016, 11:25 p.m
nformatted
> > class. In addition, you shouldn't be making subclasses as internal classes
> > of the parent abstract class.
> >
> > Di Li, when you commit code on behalf of someone else, you must make the
> > message like so: "AMBARI-X. Some message (Jo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51894/#review149466
---
Ship it!
Ship It!
- Di Li
On Sept. 16, 2016, 10:59 p.m
To reply, visit:
> https://reviews.apache.org/r/51724/
> ---
>
> (Updated Sept. 22, 2016, 12:44 p.m.)
>
>
> Review request for Ambari, Di Li, Dmytro Sen, and Sid Wagle.
>
>
> Bugs: AMBARI-17898
>
1 - 100 of 409 matches
Mail list logo