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

Íñigo Goiri commented on YARN-6526:
-----------------------------------

Thanks [~BilwaST], this looks good.
Just a couple things to wrap this up:
* I assume the unit test would give more than 1 connection before the rest of 
the patch? Are the two queries enough?
* Some of the new logs look like debug more than info, if you think they are 
not necessary move them to debug.

> Refactoring SQLFederationStateStore by avoiding to recreate a connection at 
> every call
> --------------------------------------------------------------------------------------
>
>                 Key: YARN-6526
>                 URL: https://issues.apache.org/jira/browse/YARN-6526
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: federation
>            Reporter: Giovanni Matteo Fumarola
>            Assignee: Bilwa S T
>            Priority: Major
>             Fix For: 3.4.0
>
>         Attachments: YARN-6526.001.patch, YARN-6526.002.patch, 
> YARN-6526.003.patch, YARN-6526.004.patch
>
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to