[ 
https://issues.apache.org/jira/browse/HBASE-16638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15575711#comment-15575711
 ] 

Ted Yu edited comment on HBASE-16638 at 10/14/16 4:01 PM:
----------------------------------------------------------

Integrated addendum 2 to master branch.

Thanks Weiqing


was (Author: yuzhih...@gmail.com):
>From https://builds.apache.org/job/PreCommit-HBASE-Build/4005/console :

+1 overall

| Vote |       Subsystem |  Runtime   | Comment
============================================================================
|   0  |         reexec  |  0m 14s    | Docker mode activated. 
|  +1  |      hbaseanti  |  0m 0s     | Patch does not have any anti-patterns. 
|  +1  |        @author  |  0m 0s     | The patch does not contain any @author 
|      |                 |            | tags.
|  +1  |     test4tests  |  0m 0s     | The patch appears to include 1 new or 
|      |                 |            | modified test files.
|  +1  |     mvninstall  |  3m 45s    | master passed 
|  +1  |        compile  |  1m 2s     | master passed 
|  +1  |       scaladoc  |  0m 29s    | master passed 
|  +1  |        compile  |  1m 4s     | the patch passed 
|  +1  |         scalac  |  1m 4s     | the patch passed 
|  +1  |     whitespace  |  0m 0s     | The patch has no whitespace issues. 
|  +1  |    hadoopcheck  |  27m 34s   | Patch does not cause any errors with 
|      |                 |            | Hadoop 2.4.0 2.4.1 2.5.0 2.5.1 2.5.2
|      |                 |            | 2.6.1 2.6.2 2.6.3 2.7.1.
|  +1  |    hbaseprotoc  |  0m 31s    | the patch passed 
|  +1  |       scaladoc  |  0m 22s    | the patch passed 
|  +1  |           unit  |  0m 50s    | hbase-spark in the patch passed. 
|  +1  |     asflicense  |  0m 8s     | The patch does not generate ASF License 
|      |                 |            | warnings.
|      |                 |  36m 12s   | 


|| Subsystem || Report/Notes ||
============================================================================
| Docker | Client=1.11.2 Server=1.11.2 Image:yetus/hbase:7bda515 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12833291/HBASE-16638_ADDENDUM_v2.patch
 |
| JIRA Issue | HBASE-16638 |
| Optional Tests |  asflicense  scalac  scaladoc  unit  compile  |
| uname | Linux 9aa7db47abeb 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed 
Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh
 |
| git revision | master / 91a7bbd |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/4005/testReport/ |
| modules | C: hbase-spark U: hbase-spark |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/4005/console |
| Powered by | Apache Yetus 0.3.0   http://yetus.apache.org |


> Reduce the number of Connection's created in classes of hbase-spark module
> --------------------------------------------------------------------------
>
>                 Key: HBASE-16638
>                 URL: https://issues.apache.org/jira/browse/HBASE-16638
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Weiqing Yang
>             Fix For: 2.0.0
>
>         Attachments: HBASE-16638.ADDENDUM.patch, 
> HBASE-16638_ADDENDUM_v1.patch, HBASE-16638_ADDENDUM_v2.patch, 
> HBASE-16638_v1.patch, HBASE-16638_v2.patch
>
>
> In HBaseResources.scala :
> {code}
> case class RegionResource(relation: HBaseRelation) extends ReferencedResource 
> {
>   var connection: Connection = _
>   var rl: RegionLocator = _
> ...
>   override def init(): Unit = {
>     connection = ConnectionFactory.createConnection(relation.hbaseConf)
> {code}
> Connection is created per RegionResource.
> This is not necessary.
> Caching of Connection's should be provided to reduce the number of concurrent 
> Connection's.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to