Re: Review Request 62612: Zeppelin JDBC queries should be split by default

2017-09-26 Thread venkat sairam

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62612/#review186380
---


Ship it!




Ship It!

- venkat sairam


On Sept. 27, 2017, 5:43 a.m., Prabhjyot Singh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62612/
> ---
> 
> (Updated Sept. 27, 2017, 5:43 a.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, 
> Rohit Choudhary, Sumit Mohanty, and venkat sairam.
> 
> 
> Bugs: AMBARI-22069
> https://issues.apache.org/jira/browse/AMBARI-22069
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> With ZEPPELIN-2554 there's a new interpreter setting that restricts splitting 
> of JDBC queries. This is to make "default.splitQueries=true".
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
>  8bdfaece2d 
>   ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
> 1fdbadb1e6 
> 
> 
> Diff: https://reviews.apache.org/r/62612/diff/1/
> 
> 
> Testing
> ---
> 
> Manually
> 
> 
> File Attachments
> 
> 
> AMBARI-22069_branch-2.6_v1.patch
>   
> https://reviews.apache.org/media/uploaded/files/2017/09/27/fe51e7ee-6a62-4822-a816-1062148a57b7__AMBARI-22069_branch-2.6_v1.patch
> 
> 
> Thanks,
> 
> Prabhjyot Singh
> 
>



Re: Review Request 62612: Zeppelin JDBC queries should be split by default

2017-09-26 Thread Gaurav Nagar

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62612/#review186377
---


Ship it!




Ship It!

- Gaurav Nagar


On Sept. 27, 2017, 5:43 a.m., Prabhjyot Singh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62612/
> ---
> 
> (Updated Sept. 27, 2017, 5:43 a.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, 
> Rohit Choudhary, Sumit Mohanty, and venkat sairam.
> 
> 
> Bugs: AMBARI-22069
> https://issues.apache.org/jira/browse/AMBARI-22069
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> With ZEPPELIN-2554 there's a new interpreter setting that restricts splitting 
> of JDBC queries. This is to make "default.splitQueries=true".
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
>  8bdfaece2d 
>   ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
> 1fdbadb1e6 
> 
> 
> Diff: https://reviews.apache.org/r/62612/diff/1/
> 
> 
> Testing
> ---
> 
> Manually
> 
> 
> File Attachments
> 
> 
> AMBARI-22069_branch-2.6_v1.patch
>   
> https://reviews.apache.org/media/uploaded/files/2017/09/27/fe51e7ee-6a62-4822-a816-1062148a57b7__AMBARI-22069_branch-2.6_v1.patch
> 
> 
> Thanks,
> 
> Prabhjyot Singh
> 
>



Re: Review Request 62612: Zeppelin JDBC queries should be split by default

2017-09-26 Thread Nitiraj Rathore

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62612/#review186376
---


Ship it!




Ship It!

- Nitiraj Rathore


On Sept. 27, 2017, 5:43 a.m., Prabhjyot Singh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62612/
> ---
> 
> (Updated Sept. 27, 2017, 5:43 a.m.)
> 
> 
> Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, 
> Rohit Choudhary, Sumit Mohanty, and venkat sairam.
> 
> 
> Bugs: AMBARI-22069
> https://issues.apache.org/jira/browse/AMBARI-22069
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> With ZEPPELIN-2554 there's a new interpreter setting that restricts splitting 
> of JDBC queries. This is to make "default.splitQueries=true".
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
>  8bdfaece2d 
>   ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
> 1fdbadb1e6 
> 
> 
> Diff: https://reviews.apache.org/r/62612/diff/1/
> 
> 
> Testing
> ---
> 
> Manually
> 
> 
> File Attachments
> 
> 
> AMBARI-22069_branch-2.6_v1.patch
>   
> https://reviews.apache.org/media/uploaded/files/2017/09/27/fe51e7ee-6a62-4822-a816-1062148a57b7__AMBARI-22069_branch-2.6_v1.patch
> 
> 
> Thanks,
> 
> Prabhjyot Singh
> 
>



Re: Review Request 62612: Zeppelin JDBC queries should be split by default

2017-09-26 Thread Prabhjyot Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62612/
---

(Updated Sept. 27, 2017, 11:13 a.m.)


Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, Rohit 
Choudhary, Sumit Mohanty, and venkat sairam.


Bugs: AMBARI-22069
https://issues.apache.org/jira/browse/AMBARI-22069


Repository: ambari


Description
---

With ZEPPELIN-2554 there's a new interpreter setting that restricts splitting 
of JDBC queries. This is to make "default.splitQueries=true".


Diffs
-

  
ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
 8bdfaece2d 
  ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
1fdbadb1e6 


Diff: https://reviews.apache.org/r/62612/diff/1/


Testing
---

Manually


File Attachments (updated)


AMBARI-22069_branch-2.6_v1.patch
  
https://reviews.apache.org/media/uploaded/files/2017/09/27/fe51e7ee-6a62-4822-a816-1062148a57b7__AMBARI-22069_branch-2.6_v1.patch


Thanks,

Prabhjyot Singh



Review Request 62612: Zeppelin JDBC queries should be split by default

2017-09-26 Thread Prabhjyot Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62612/
---

Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Jayush Luniya, Rohit 
Choudhary, Sumit Mohanty, and venkat sairam.


Bugs: AMBARI-22069
https://issues.apache.org/jira/browse/AMBARI-22069


Repository: ambari


Description
---

With ZEPPELIN-2554 there's a new interpreter setting that restricts splitting 
of JDBC queries. This is to make "default.splitQueries=true".


Diffs
-

  
ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
 8bdfaece2d 
  ambari-server/src/test/python/stacks/2.6/ZEPPELIN/test_zeppelin_070.py 
1fdbadb1e6 


Diff: https://reviews.apache.org/r/62612/diff/1/


Testing
---

Manually


Thanks,

Prabhjyot Singh