Íñ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

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