/main/resources/stacks/HDP/2.6/services/TEZ/configuration/tez-site.xml
393d077eb7
Diff: https://reviews.apache.org/r/64276/diff/1/
Testing
---
Tested manually on repro cluster.
Since only tez-site.xml file is changed, unit tests are not affected
Thanks,
Jaimin Jetly
bits was removed from trunk bits
Diffs
-
ambari-web/app/controllers/wizard/step8_controller.js b99c6e39d7
Diff: https://reviews.apache.org/r/62427/diff/1/
Testing
---
ambari-web unit test:
21334 passing (27s)
128 pending
Thanks,
Jaimin Jetly
Thanks,
Jaimin Jetly
://reviews.apache.org/r/62294/diff/1/
Testing (updated)
---
Tested manually on a cluster.
verified that ambari-web unit test and ExecutionScheduleManagerTest test passes
successfully:
30490 passing (20s)
157 pending
Thanks,
Jaimin Jetly
:
30489 passing (20s)
157 pending
Thanks,
Jaimin Jetly
e0c2f5d1d0
Diff: https://reviews.apache.org/r/62235/diff/1/
Testing
---
Manually tested that the patch fixes the issue.
ambari-web unit test result:
30489 passing (20s)
157 pending
Thanks,
Jaimin Jetly
/reassign/step1_controller.js
c631e85bcb
Diff: https://reviews.apache.org/r/61894/diff/1/
Testing
---
Verified e2e that patch fixes the issue.
Verified that ambari-web unit tests passes with the patch
21208 passing (22s)
128 pending
Thanks,
Jaimin Jetly
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60516/#review179165
---
Ship it!
Ship It!
- Jaimin Jetly
On June 28, 2017, 8:31 p.m
ified behavior
is as expected
Thanks,
Jaimin Jetly
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59514/#review175904
---
Ship it!
Ship It!
- Jaimin Jetly
On May 24, 2017, 6:15 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59427/#review175744
---
Ship it!
Ship It!
- Jaimin Jetly
On May 23, 2017, 1:49 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59336/#review175249
---
Ship it!
Ship It!
- Jaimin Jetly
On May 17, 2017, 2:10 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59315/#review175126
---
Ship it!
Ship It!
- Jaimin Jetly
On May 16, 2017, 5:32 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59236/#review174838
---
Ship it!
Ship It!
- Jaimin Jetly
On May 12, 2017, 6:59 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59187/#review174739
---
Ship it!
Ship It!
- Jaimin Jetly
On May 11, 2017, 10:07 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59104/#review174348
---
Ship it!
Ship It!
- Jaimin Jetly
On May 9, 2017, 5:50 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59043/#review174170
---
Ship it!
Ship It!
- Jaimin Jetly
On May 8, 2017, 7 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58637/#review172725
---
Ship it!
Ship It!
- Jaimin Jetly
On April 21, 2017, 11:55
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58596/#review172567
---
Ship it!
Ship It!
- Jaimin Jetly
On April 21, 2017, 12:09
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58433/#review171951
---
Ship it!
Ship It!
- Jaimin Jetly
On April 13, 2017, 10:08
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58356/#review171884
---
Ship it!
Ship It!
- Jaimin Jetly
On April 13, 2017, 3:44
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58405/#review171825
---
Ship it!
Ship It!
- Jaimin Jetly
On April 12, 2017, 10:19
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58405/#review171824
---
Ship it!
Ship It!
- Jaimin Jetly
On April 12, 2017, 10:19
> On April 11, 2017, 2:51 a.m., Jaimin Jetly wrote:
> > Enabling interactive query for Hive which installs HiveServer2 interactive
> > host component and deploys LLAP requires slider service.
> > That's the reason for slider being as a dependency for Hive.
> >
>
vices" page for hive
Reference: https://cwiki.apache.org/confluence/display/Hive/LLAP
- Jaimin Jetly
On April 11, 2017, 1:05 a.m., zhangxiaolu zhangxiaolu wrote:
>
> ---
> This is an automatically generated e-mail. To re
that the swagger spec file has no validation error.
verified that html doc file is generated as expected.
Thanks,
Jaimin Jetly
obs ... SKIPPED
[INFO] Pig .... SKIPPED
- Jaimin Jetly
On March 24, 2017, 5:25 p.m., venkat sairam wrote:
>
> ---
> This is an automa
NFO]
Thanks,
Jaimin Jetly
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57858/#review169836
---
Ship it!
Ship It!
- Jaimin Jetly
On March 22, 2017, 6:06
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57834/#review169759
---
Ship it!
Ship It!
- Jaimin Jetly
On March 22, 2017, 9:46
t;INSTALLED", "NOT_INSTALLED" and "NOT_REQUIRED"?
- Jaimin Jetly
On March 22, 2017, 9:44 a.m., Attila Magyar wrote:
>
> ---
> This is an automatically generated e-mail. To reply
3062-3066 (patched)
<https://reviews.apache.org/r/57813/#comment242039>
only if '' param is present in the url then lets append opt.url =
'&_=' + App.datetime instead of '?_='
- Jaimin Jetly
On March 21,
e.
The package name issue has been addressed in the last revision of the patch
- Jaimin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57582/#review169161
-------
RI-20508) to do analysis on how much
of the code we can use from the vendor library instead of maintaining all in
ambari-web source code.
- Jaimin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apach
-
Tested on dev environment for all integrated apis to be working from ui
interface
Verified that Rat check, style check and existing ambari-server unit tests are
passing with the patch
Thanks,
Jaimin Jetly
to work e2e then please
commit it.
- Jaimin Jetly
On March 16, 2017, 12:49 p.m., Oleg Nechiporenko wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
290-295 (original), 290-297 (patched)
<https://reviews.apache.org/r/57507/#comment241545>
will this have any UX impact ?
- Jaimin Jetly
On March 16, 2017, 12:49 p.m., Oleg Nechiporenko wrote:
>
> ---
> This is a
successfully. Also build other ambari ui projects and verified that others are
not impacted with this issue
Thanks,
Jaimin Jetly
ps://reviews.apache.org/r/57582/#review168931
---
On March 14, 2017, 11:35 p.m., Jaimin Jetly wrote:
>
> ---
> This is an automatically generated e-mail.
ll integrated apis to be working from ui
interface
Verified that Rat check, style check and existing ambari-server unit tests are
passing with the patch
Thanks,
Jaimin Jetly
-CREATION
ambari-web/brunch-config.js 31eb1cb
Diff: https://reviews.apache.org/r/57582/diff/1/
Testing
---
Tested on dev environment for all integrated apis to be working from ui
interface
Verified that Rat check, style check and existing ambari-server unit tests are
passing with the patch
Thanks,
Jaimin Jetly
/brunch-config.js 31eb1cb
Diff: https://reviews.apache.org/r/57582/diff/1/
Testing
---
Tested on dev environment for all integrated apis to be working from ui
interface
Verified that Rat check, style check and existing ambari-server unit tests are
passing with the patch
Thanks,
Jaimin Jetly
ace
Verified that Rat check, style check and existing ambari-server unit tests are
passing with the patch
Thanks,
Jaimin Jetly
--
Tested the patch on a repro cluster.
veified that all ambari-web unit tests passes:
30367 passing (21s)
157 pending
Thanks,
Jaimin Jetly
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57567/#review168824
---
Ship it!
Ship It!
- Jaimin Jetly
On March 13, 2017, 7:20
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57526/#review168695
---
Ship it!
Ship It!
- Jaimin Jetly
On March 11, 2017, 12:48
(original), 84-87 (patched)
<https://reviews.apache.org/r/57268/#comment239918>
Whats the reason behind making this computed property volatile ?
- Jaimin Jetly
On March 3, 2017, 7:34 a.m., Richard Zang
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57062/#review167136
---
Ship it!
Ship It!
- Jaimin Jetly
On Feb. 28, 2017, 7:44 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57062/#review166985
---
Ship it!
Ship It!
- Jaimin Jetly
On Feb. 25, 2017, 4:34 a.m
tps://reviews.apache.org/r/57004/#comment238630>
$.merge(response, App.get('supports')) returns back the first argument
passed. As a result the application behavior remains the same as it was before
this patch
Please use $.extend instead of $.merge.
- Jaimin Jetly
On Feb. 23, 2017,
enhancement of
making sure both the component and host scope depencdency by default appears on
the same host can be achieved by addressing
https://issues.apache.org/jira/browse/AMBARI-20122
- Jaimin Jetly
On Feb. 22, 2017, 1:52 a.m., yao lei wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56597/#review165592
---
Ship it!
Ship It!
- Jaimin Jetly
On Feb. 13, 2017, 11:50
> > This could be problematic if the merge fails when the transaction
> > commits. You've already fired the event.
>
> Jaimin Jetly wrote:
> persistAction method has been annotated with @Transactional and
> @TransactionalLock.
> I believe if the publisher met
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56426/#review164761
---
Ship it!
Ship It!
- Jaimin Jetly
On Feb. 8, 2017, 8:18 p.m
ficant. so I am ok with commiting this patch.
But good to track this as a future requirement for ambari-server to expose an
API that gives sorted hostnames for a configgroup by default or via sortBy
predicate.
- Jaimin Jetly
On Feb. 8, 2017, 3:34 a.m.,
ransaction, so if merging of
stage/request status fails in the subscriber method of the taskupdate event
then rollback happens for hostrolecommand status being updated from
HostRoleCommandDAO#mergeWithoutPublishEvent method.
- Jaimin
for these operations. There was a
minor performance gain with the patch. As part of
https://issues.apache.org/jira/browse/AMBARI-18889, I will look if we can use
the request status and stage status to further enhance performance.
Thanks,
Jaimin Jetly
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56337/#review164397
---
Ship it!
Ship It!
- Jaimin Jetly
On Feb. 6, 2017, 7:24 p.m
7f047c8
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java
8a35c98
Diff: https://reviews.apache.org/r/56233/diff/
Testing
---
Tested on slower machines.
Hadoop QA job on the jira stated no existing test failures
Thanks,
Jaimin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56198/#review163919
---
Ship it!
Ship It!
- Jaimin Jetly
On Feb. 2, 2017, 12:58 a.m
eviews.apache.org/r/56097/#review163691
---
On Jan. 30, 2017, 9:30 p.m., Jaimin Jetly wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56097/
> ---
---
Verified on manual cluster
verified that all ambari-web unit tests passes:
20325 passing (24s)
153 pending
Thanks,
Jaimin Jetly
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55953/#review163535
---
Ship it!
Ship It!
- Jaimin Jetly
On Jan. 29, 2017, 5:11 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56033/#review163360
---
Ship it!
Ship It!
- Jaimin Jetly
On Jan. 27, 2017, 7:25 p.m
been tested with wizards ?
- Jaimin Jetly
On Jan. 27, 2017, 7:25 p.m., Xi Wang wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
ope
https://reviews.apache.org/media/uploaded/files/2017/01/24/c942ca85-6c5a-4ed0-beee-f0ce16b91532__Validation_error_host_scope.png
Thanks,
Jaimin Jetly
tps://reviews.apache.org/media/uploaded/files/2017/01/24/c942ca85-6c5a-4ed0-beee-f0ce16b91532__Validation_error_host_scope.png
Thanks,
Jaimin Jetly
org/media/uploaded/files/2017/01/24/c942ca85-6c5a-4ed0-beee-f0ce16b91532__Validation_error_host_scope.png
Thanks,
Jaimin Jetly
ork with the patch as expected.
snapshots attached on the jira of the UI impact
Verfiied that related ambari-server (java/python) unit test passes with the
patch
Thanks,
Jaimin Jetly
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55700/#review162348
---
Ship it!
Ship It!
- Jaimin Jetly
On Jan. 19, 2017, 2:39 a.m
On Dec. 15, 2016, 4:49 p.m., Jaimin Jetly wrote:
> > One thing I don't quite see here (and it could be due to the size of the
> > patch) is what happens in these two cases:
> > - Something goes wrong when trying to store a task's status. How does the
> > system recov
> > This could be problematic if the merge fails when the transaction
> > commits. You've already fired the event.
>
> Jaimin Jetly wrote:
> persistAction method has been annotated with @Transactional and
> @TransactionalLock.
> I believe if the publisher met
, I will look if we can use
the request status and stage status to further enhance performance.
Thanks,
Jaimin Jetly
meant was this isn't a typically-named java variable. Just use
> > something like "taskStatusLoaded", and make the field private.
>
> Jaimin Jetly wrote:
> ok. I will address this in next revision of the patch.
This has been addressed in new revision of the patch.
&
will look if we can use
the request status and stage status to further enhance performance.
Thanks,
Jaimin Jetly
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55399/#review161304
---
Ship it!
Ship It!
- Jaimin Jetly
On Jan. 11, 2017, 12:34
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53213/#review161291
---
Ship it!
Ship It!
- Jaimin Jetly
On Nov. 9, 2016, 12:11 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55367/#review161106
---
Ship it!
Ship It!
- Jaimin Jetly
On Jan. 10, 2017, 2:37 a.m
59315
-------
On Dec. 14, 2016, 8:20 p.m., Jaimin Jetly wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/53686/
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55233/#review160643
---
Ship it!
Ship It!
- Jaimin Jetly
On Jan. 5, 2017, 10:18 p.m
/main/host/details_test.js 154d0b2
ambari-web/test/controllers/main/service/item_test.js 082fbd9
Diff: https://reviews.apache.org/r/55163/diff/
Testing
---
ambari-web unit test result with the patch:
29792 passing (20s)
154 pending
Thanks,
Jaimin Jetly
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55096/#review160302
---
Ship it!
Ship It!
- Jaimin Jetly
On Dec. 30, 2016, 3:41 a.m
t;
> > Do you depend on order of execution for success? If yes, this looks
> > non-reliable.
>
> Jaimin Jetly wrote:
> Not for success. But this order helps in collecting entire error/output
> stream data.
> We first start thread that collects the error/output
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55062/#review160250
---
Ship it!
Ship It!
- Jaimin Jetly
On Dec. 28, 2016, 2:52 a.m
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54958/#review160172
-------
On Dec. 27, 2016, 4:16 p.m., Jaimin Jetly wrote:
>
>
t:
https://reviews.apache.org/r/54958/#review160170
-------
On Dec. 27, 2016, 4:16 p.m., Jaimin Jetly wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54958/
> ---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54958/#review159970
---
On Dec. 23, 2016, 12:57 a.m., Jaimin Jetly wrote:
>
> ---
- Jaimin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54958/#review159924
---
On Dec. 23, 2016, 12:57 a.m., Jaimin Jetly wrote:
>
> ---
&
there is no regression
Jenkins QA job for 1st revision of the patch compltedt successsfully
Jenkins QA job for 2nd revision of the patch in progress...
Thanks,
Jaimin Jetly
all unit tests on dev machine and checked there is no regression
Jenkins QA job for unit test in progress...
Thanks,
Jaimin Jetly
/diff/
Testing
---
Tested manually on a cluster
Added validation to existing unit tests
Executed all unit tests on dev machine and checked there is no regression
Jenkins QA job for unit test in progress...
Thanks,
Jaimin Jetly
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54460/#review159645
---
Ship it!
Ship It!
- Jaimin Jetly
On Dec. 19, 2016, 7:38 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54733/#review159203
---
Ship it!
Ship It!
- Jaimin Jetly
On Dec. 14, 2016, 8:46 a.m
eviews.apache.org/r/53686/#review155819
---
On Dec. 14, 2016, 2:11 a.m., Jaimin Jetly wrote:
>
> ---
> This is an automatically generated e-mail. To re
node simulated cluster
- Jaimin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53686/#review155819
---
On Dec. 14, 20
can use
the request status and stage status to further enhance performance.
Thanks,
Jaimin Jetly
/disable_controller.js (line 26)
<https://reviews.apache.org/r/54698/#comment229956>
It is possible that zookeeper is alreay stopped when user starts
de-kerberizing. To handle that situation additional step to start zookeeper
will be required.
['stopServices', 'startZk',...]
- Jaimin Jetly
On D
s.apache.org/r/54644/diff/
Testing (updated)
---
Verified that all ambari-web unit tests run without getting short-circuited at
anypoint:
Verified that ambari-web unit test runs on Mac OS X El Capitan, Mac OS Sierra
and Ubuntu.
npm test:
29760 passing (20s)
154 pending
Thanks,
Jaimin Jetly
s.apache.org/r/54644/diff/
Testing
---
Verified that all ambari-web unit tests run:
Verified that ambari-web unit test runs on Mac OS X El Capitan, Mac OS Sierra
and Ubuntu.
npm test:
29760 passing (20s)
154 pending
Thanks,
Jaimin Jetly
i-web unit tests run:
Verified that ambari-web unit test runs on Mac OS X El Capitan, Mac OS Sierra
and Ubuntu.
npm test:
29760 passing (20s)
154 pending
Thanks,
Jaimin Jetly
1 - 100 of 217 matches
Mail list logo