, bundle etc)
and some further refactoring (the use of validator etc). Let me review thi a
bit more carefully if there are intersections
- Venkat Ranganathan
On Oct. 21, 2016, 11:28 a.m., Sangeeta Ravindran wrote:
>
> ---
> T
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51863/#review148807
---
Ship it!
Ship It!
- Venkat Ranganathan
On Sept. 13, 2016, 2
ld have troubles with customized
> users/permissions/non root user.
>
> Venkat Ranganathan wrote:
> Yes. This is only one and it is really not a JDBC driver. We just use
> the download mechanism
>
> Vitalyi Brodetskyi wrote:
> Ok, but anyway i think it will be go
il. To reply, visit:
https://reviews.apache.org/r/50595/#review144469
-----------
On Aug. 1, 2016, 2:21 p.m., Venkat Ranganathan wrote:
>
> ---
> This is an automatically generat
file
- Venkat
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51154/#review145917
---
On Aug. 16, 2016, 4:19 p.m., Venkat Ra
Total failures:0
OK
Thanks,
Venkat Ranganathan
failures:0
OK
Thanks,
Venkat Ranganathan
/package/scripts/oozie.py
(line 378)
<https://reviews.apache.org/r/51121/#comment212258>
we need to kinit if needed here
- Venkat Ranganathan
On Aug. 15, 2016, 5:40 p.m., Alejandro Fernandez wrote:
>
> ---
> This is a
/package/scripts/oozie.py
(line 313)
<https://reviews.apache.org/r/51121/#comment212256>
Currently we don't support propeties file for action config files. This
needs to go to sharelib also
- Venkat Ranganathan
On Aug. 15, 2016, 5:40 p.m., Alejandro Fern
/package/scripts/oozie.py
(line 410)
<https://reviews.apache.org/r/51121/#comment212255>
We should do this before Oozie is started or you should run the following
command
oozie admin -oozie -sharelibupdate
- Venkat Ranganathan
On Aug. 15, 2016, 5:40 p.m., Alejandro Fer
/configuration/hive-site.xml
(line 24)
<https://reviews.apache.org/r/51121/#comment212231>
Why do we need this file?
- Venkat Ranganathan
On Aug. 15, 2016, 5:40 p.m., Alejandro Fernandez wrote:
>
> ---
> This is a
To reply, visit:
https://reviews.apache.org/r/50595/#review144468
-------
On Aug. 1, 2016, 2:21 p.m., Venkat Ranganathan wrote:
>
> ---
> This is an automatical
://reviews.apache.org/r/50691/diff/
Testing
---
All editors were tested and validated with a Oozie server
Thanks,
Venkat Ranganathan
Thanks,
Venkat Ranganathan
: https://reviews.apache.org/r/50595/diff/
Testing
---
Tested downloading of the jar file by registering with ambari server
Tested Falcon/Atlas component names
Thanks,
Venkat Ranganathan
downloading of the jar file by registering with ambari server
Tested Falcon/Atlas component names
Thanks,
Venkat Ranganathan
file by registering with ambari server
Tested Falcon/Atlas component names
Thanks,
Venkat Ranganathan
eviews.apache.org/r/50595/#review144291
---
On Aug. 1, 2016, 10:38 a.m., Venkat Ranganathan wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/50595/
> ---
names
Thanks,
Venkat Ranganathan
Falcon/Atlas component names
Thanks,
Venkat Ranganathan
line 211
> > <https://reviews.apache.org/r/50595/diff/1/?file=1457507#file1457507line211>
> >
> > Typo.
will fix. thanks
- Venkat
---
This is an automatically generated e-mail. To reply, visit:
http
component names
Thanks,
Venkat Ranganathan
/stacks/HDP/2.5/services/stack_advisor.py
f8ba3e9
Diff: https://reviews.apache.org/r/50595/diff/
Testing
---
Tested downloading of the jar file by registering with ambari server
Tested Falcon/Atlas component names
Thanks,
Venkat Ranganathan
f8ba3e9
Diff: https://reviews.apache.org/r/50431/diff/
Testing
---
Thanks,
Venkat Ranganathan
://reviews.apache.org/r/50431/diff/
Testing
---
Thanks,
Venkat Ranganathan
Diff: https://reviews.apache.org/r/50431/diff/
Testing
---
Thanks,
Venkat Ranganathan
ebe3634
ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py
315020b
Diff: https://reviews.apache.org/r/50431/diff/
Testing
---
Thanks,
Venkat Ranganathan
> On July 21, 2016, 8:56 a.m., Venkat Ranganathan wrote:
> > ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py,
> > line 331
> > <https://reviews.apache.org/r/50287/diff/1/?file=1449672#file1449672line331>
> >
> >
/diff/
Testing
---
Validated the config
Thanks,
Venkat Ranganathan
/services/stack_advisor.py
315020b
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/FixOozieAdminUsersTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/50256/diff/
Testing
---
Added UT and successfully executed it
Thanks,
Venkat Ranganathan
/stack_advisor.py (line
331)
<https://reviews.apache.org/r/50287/#comment208799>
Do we need this. Isn't OOZIE added in HDP21StackAdvisor whose dictionary
is merged in succeeding child classes?
- Venkat Ranganathan
On July 21, 2016, 3:19 a.m., Mugdha Var
/upgrades/FixOozieAdminUsersTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/50256/diff/
Testing
---
Added UT and successfully executed it
Thanks,
Venkat Ranganathan
/50256/diff/
Testing
---
Added UT and successfully executed it
Thanks,
Venkat Ranganathan
ll create a new bug to track
this
- Venkat
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50178/#review142779
-------
On July
: https://reviews.apache.org/r/50178/diff/
Testing
---
Implemented the changes in stack_advisor and ran a test to make sure the
configuration changes for admin user is forced
Thanks,
Venkat Ranganathan
eviews.apache.org/r/49222/#review139471
---
On June 24, 2016, 6:27 p.m., Venkat Ranganathan wrote:
>
> ---
> This is an automatically generated e-mail. To re
/src/main/resources/common-services/FALCON/0.5.0.2.1/package/scripts/falcon.py
6cb4089
ambari-server/src/main/resources/stacks/HDP/2.5/services/FALCON/configuration/oozie-site.xml
PRE-CREATION
Diff: https://reviews.apache.org/r/49222/diff/
Testing
---
Thanks,
Venkat Ranganathan
-----
On June 23, 2016, 12:05 a.m., Venkat Ranganathan wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48609/
>
ambari-server/src/test/python/stacks/2.2/configs/falcon-upgrade.json 7499cdf
Diff: https://reviews.apache.org/r/48609/diff/
Testing (updated)
---
UT passes
Thanks,
Venkat Ranganathan
, visit:
https://reviews.apache.org/r/48609/#review137377
-----------
On June 15, 2016, 10:51 a.m., Venkat Ranganathan wrote:
>
> ---
> This is an automatically gener
/FALCON/package/scripts/falcon.py
9a72af1
ambari-server/src/main/resources/stacks/HDP/2.3/services/FALCON/configuration/falcon-client.properties.xml
PRE-CREATION
Diff: https://reviews.apache.org/r/48609/diff/
Testing
---
Thanks,
Venkat Ranganathan
/services/FALCON/configuration/falcon-client.properties.xml
PRE-CREATION
Diff: https://reviews.apache.org/r/48609/diff/
Testing
---
Thanks,
Venkat Ranganathan
://reviews.apache.org/r/48355/diff/
Testing
---
Made sure the startup properties changes are valid manually
Thanks,
Venkat Ranganathan
/2.5/services/FALCON/configuration/falcon-startup.properties.xml
0f542cc
Diff: https://reviews.apache.org/r/48355/diff/
Testing
---
Made sure the startup properties changes are valid manually
Thanks,
Venkat Ranganathan
/config-upgrade.xml
60cac05
ambari-server/src/main/resources/stacks/HDP/2.5/services/FALCON/configuration/falcon-startup.properties.xml
0f542cc
Diff: https://reviews.apache.org/r/48355/diff/
Testing
---
Made sure the startup properties changes are valid manually
Thanks,
Venkat
-server/src/main/resources/stacks/HDP/2.5/services/FALCON/configuration/falcon-startup.properties.xml
0f542cc
Diff: https://reviews.apache.org/r/48355/diff/
Testing (updated)
---
Made sure the startup properties changes are valid manually
Thanks,
Venkat Ranganathan
-startup.properties.xml
0f542cc
Diff: https://reviews.apache.org/r/48355/diff/
Testing
---
Thanks,
Venkat Ranganathan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47009/#review132824
---
Ship it!
Ship It!
- Venkat Ranganathan
On May 11, 2016, 1
> On May 9, 2016, 2:49 p.m., Venkat Ranganathan wrote:
> >
We have committed a change as part of FALCON-1942 to add a environment variable
called FALCON_EXTRA_CLASS_PATH - this will be appended to the Falcon server
classpath. Please use it
> On May 9, 2016, 2:49 p.m., Venkat
/0.5.0.2.1/configuration/falcon-env.xml
(line 162)
<https://reviews.apache.org/r/47009/#comment196515>
Don't we want the server classpath to be changed in falcon-config?
- Venkat Ranganathan
On May 5, 2016, 5:07 a.m., Tom Beer
50 matches
Mail list logo