You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Alexandr Antonenko <hi...@gmail.com> on 2015/03/19 18:29:12 UTC

Review Request 32243: Hive View: Maintain one session per tab

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

Review request for Ambari, Andriy Babiichuk and Aleksandr Kovalenko.


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


Repository: ambari


Description
-------

Hive View should use same session for all queries executed from same tab.
Fixes DB explorer issue


Diffs
-----

  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/Connection.java 44c90ae 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/DDLDelegator.java 9cb5a1b 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/CRUDResourceManager.java 28cf4d1 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/PersonalCRUDResourceManager.java 15fcf22 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/browser/HiveBrowserService.java bb1a0a2 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/Aggregator.java ce82e15 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/ConnectionController.java b70abe2 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/JobService.java d9c69e8 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/OperationHandleResourceManager.java 5004677 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/atsJobs/ATSParser.java b644d4c 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/atsJobs/IATSParser.java d029fdc 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/atsJobs/TezDagId.java 061c51c 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/Job.java 004932c 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobControllerImpl.java a100f3d 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobImpl.java 1858b3e 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobResourceManager.java 101e328 
  contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/savedQueries/SavedQueryResourceManager.java c032bb1 
  contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/index.js 592bb06 
  contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/settings.js c59fc88 
  contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/tez-ui.js dc99fd1 
  contrib/views/hive/src/main/resources/ui/hive-web/app/initializers/i18n.js a87bf78 
  contrib/views/hive/src/main/resources/ui/hive-web/app/models/job.js 73fcc20 
  contrib/views/hive/src/main/resources/ui/hive-web/app/styles/app.scss 7085c85 
  contrib/views/hive/src/main/resources/ui/hive-web/app/templates/index.hbs b8d1d3b 
  contrib/views/hive/src/main/resources/ui/hive-web/app/templates/tez-ui.hbs f9b8d9c 
  contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/ATSParserTest.java ced7772 
  contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/AggregatorTest.java 08e8b81 
  contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/JobServiceTest.java 1c3444e 

Diff: https://reviews.apache.org/r/32243/diff/


Testing
-------


Thanks,

Alexandr Antonenko


Re: Review Request 32243: Hive View: Maintain one session per tab

Posted by Aleksandr Kovalenko <ak...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32243/#review77070
-----------------------------------------------------------

Ship it!


Ship It!

- Aleksandr Kovalenko


On Березень 19, 2015, 5:29 після полудня, Alexandr Antonenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32243/
> -----------------------------------------------------------
> 
> (Updated Березень 19, 2015, 5:29 після полудня)
> 
> 
> Review request for Ambari, Andriy Babiichuk and Aleksandr Kovalenko.
> 
> 
> Bugs: AMBARI-10132
>     https://issues.apache.org/jira/browse/AMBARI-10132
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Hive View should use same session for all queries executed from same tab.
> Fixes DB explorer issue
> 
> 
> Diffs
> -----
> 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/Connection.java 44c90ae 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/DDLDelegator.java 9cb5a1b 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/CRUDResourceManager.java 28cf4d1 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/PersonalCRUDResourceManager.java 15fcf22 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/browser/HiveBrowserService.java bb1a0a2 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/Aggregator.java ce82e15 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/ConnectionController.java b70abe2 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/JobService.java d9c69e8 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/OperationHandleResourceManager.java 5004677 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/atsJobs/ATSParser.java b644d4c 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/atsJobs/IATSParser.java d029fdc 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/atsJobs/TezDagId.java 061c51c 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/Job.java 004932c 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobControllerImpl.java a100f3d 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobImpl.java 1858b3e 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobResourceManager.java 101e328 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/savedQueries/SavedQueryResourceManager.java c032bb1 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/index.js 592bb06 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/settings.js c59fc88 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/tez-ui.js dc99fd1 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/initializers/i18n.js a87bf78 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/models/job.js 73fcc20 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/styles/app.scss 7085c85 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/templates/index.hbs b8d1d3b 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/templates/tez-ui.hbs f9b8d9c 
>   contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/ATSParserTest.java ced7772 
>   contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/AggregatorTest.java 08e8b81 
>   contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/JobServiceTest.java 1c3444e 
> 
> Diff: https://reviews.apache.org/r/32243/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alexandr Antonenko
> 
>


Re: Review Request 32243: Hive View: Maintain one session per tab

Posted by Andriy Babiichuk <ab...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32243/#review77069
-----------------------------------------------------------

Ship it!


Ship It!

- Andriy Babiichuk


On Март 19, 2015, 5:29 п.п., Alexandr Antonenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32243/
> -----------------------------------------------------------
> 
> (Updated Март 19, 2015, 5:29 п.п.)
> 
> 
> Review request for Ambari, Andriy Babiichuk and Aleksandr Kovalenko.
> 
> 
> Bugs: AMBARI-10132
>     https://issues.apache.org/jira/browse/AMBARI-10132
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Hive View should use same session for all queries executed from same tab.
> Fixes DB explorer issue
> 
> 
> Diffs
> -----
> 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/Connection.java 44c90ae 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/DDLDelegator.java 9cb5a1b 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/CRUDResourceManager.java 28cf4d1 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/PersonalCRUDResourceManager.java 15fcf22 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/browser/HiveBrowserService.java bb1a0a2 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/Aggregator.java ce82e15 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/ConnectionController.java b70abe2 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/JobService.java d9c69e8 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/OperationHandleResourceManager.java 5004677 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/atsJobs/ATSParser.java b644d4c 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/atsJobs/IATSParser.java d029fdc 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/atsJobs/TezDagId.java 061c51c 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/Job.java 004932c 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobControllerImpl.java a100f3d 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobImpl.java 1858b3e 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobResourceManager.java 101e328 
>   contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/savedQueries/SavedQueryResourceManager.java c032bb1 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/index.js 592bb06 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/settings.js c59fc88 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/tez-ui.js dc99fd1 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/initializers/i18n.js a87bf78 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/models/job.js 73fcc20 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/styles/app.scss 7085c85 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/templates/index.hbs b8d1d3b 
>   contrib/views/hive/src/main/resources/ui/hive-web/app/templates/tez-ui.hbs f9b8d9c 
>   contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/ATSParserTest.java ced7772 
>   contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/AggregatorTest.java 08e8b81 
>   contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/JobServiceTest.java 1c3444e 
> 
> Diff: https://reviews.apache.org/r/32243/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alexandr Antonenko
> 
>