[jira] [Comment Edited] (HIVE-11891) Add basic performance logging at trace level to metastore calls
[ https://issues.apache.org/jira/browse/HIVE-11891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14876773#comment-14876773 ] Brock Noland edited comment on HIVE-11891 at 9/19/15 1:03 AM: -- I was looking at it again and I think it can be done by moving most {{getPerfLogger}} calls to {{SessionState}}. Updated patch attached. Note this patch logs at {{INFO}} since that is what PerfLogger does by default. I think that is OK since the HMS logs a few messages per call to the HMS anyway. Example output: {noformat} 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogBegin(117)) - 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogEnd(157)) - {noformat} was (Author: brocknoland): I was looking at it again and I think it can be done by moving most {{getPerfLogger}} calls to {{SessionState}}. Updated patch attached. Note this patch logs at {{LOG}} since that is what PerfLogger does by default. I think that is OK since the HMS logs a few messages per call to the HMS anyway. Example output: {noformat} 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogBegin(117)) - 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogEnd(157)) - {noformat} > Add basic performance logging at trace level to metastore calls > --- > > Key: HIVE-11891 > URL: https://issues.apache.org/jira/browse/HIVE-11891 > Project: Hive > Issue Type: Improvement > Components: Metastore >Affects Versions: 1.0.0, 1.2.0, 1.1.0 >Reporter: Brock Noland >Assignee: Brock Noland >Priority: Minor > Fix For: 2.0.0 > > Attachments: HIVE-11891.patch, HIVE-11891.patch, HIVE-11891.patch > > > At present it's extremely difficult to debug slow calls to the metastore. > Ideally there would be some basic means of doing so. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Comment Edited] (HIVE-11891) Add basic performance logging at trace level to metastore calls
[ https://issues.apache.org/jira/browse/HIVE-11891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14876773#comment-14876773 ] Brock Noland edited comment on HIVE-11891 at 9/19/15 1:04 AM: -- I was looking at it again and I think it can be done by moving most {{getPerfLogger}} calls to {{SessionState}}. Updated patch attached. Note this patch logs at {{INFO}} since that is what PerfLogger does by default. I think that is OK since the HMS logs a few messages per call to the HMS anyway. Example output: {noformat} 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogBegin(117)) - 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogEnd(157)) - {noformat} The big changes are in {{PerfLogger,SessionState. getPerfLogger}} was (Author: brocknoland): I was looking at it again and I think it can be done by moving most {{getPerfLogger}} calls to {{SessionState}}. Updated patch attached. Note this patch logs at {{INFO}} since that is what PerfLogger does by default. I think that is OK since the HMS logs a few messages per call to the HMS anyway. Example output: {noformat} 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogBegin(117)) - 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogEnd(157)) - {noformat} > Add basic performance logging at trace level to metastore calls > --- > > Key: HIVE-11891 > URL: https://issues.apache.org/jira/browse/HIVE-11891 > Project: Hive > Issue Type: Improvement > Components: Metastore >Affects Versions: 1.0.0, 1.2.0, 1.1.0 >Reporter: Brock Noland >Assignee: Brock Noland >Priority: Minor > Fix For: 2.0.0 > > Attachments: HIVE-11891.patch, HIVE-11891.patch, HIVE-11891.patch > > > At present it's extremely difficult to debug slow calls to the metastore. > Ideally there would be some basic means of doing so. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Comment Edited] (HIVE-11891) Add basic performance logging at trace level to metastore calls
[ https://issues.apache.org/jira/browse/HIVE-11891?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14876773#comment-14876773 ] Brock Noland edited comment on HIVE-11891 at 9/19/15 1:07 AM: -- I was looking at it again and I think it can be done by moving most {{getPerfLogger}} calls to {{SessionState}}. Updated patch attached. Note this patch logs at {{INFO}} since that is what PerfLogger does by default. I think that is OK since the HMS logs a few messages per call to the HMS anyway. Example output: {noformat} 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogBegin(117)) - 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogEnd(157)) - {noformat} The big changes are in {{PerfLogger,SessionState.getPerfLogger}} was (Author: brocknoland): I was looking at it again and I think it can be done by moving most {{getPerfLogger}} calls to {{SessionState}}. Updated patch attached. Note this patch logs at {{INFO}} since that is what PerfLogger does by default. I think that is OK since the HMS logs a few messages per call to the HMS anyway. Example output: {noformat} 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogBegin(117)) - 2015-09-18T18:01:04,389 INFO [pool-6-thread-1]: log.PerfLogger (PerfLogger.java:PerfLogEnd(157)) - {noformat} The big changes are in {{PerfLogger,SessionState. getPerfLogger}} > Add basic performance logging at trace level to metastore calls > --- > > Key: HIVE-11891 > URL: https://issues.apache.org/jira/browse/HIVE-11891 > Project: Hive > Issue Type: Improvement > Components: Metastore >Affects Versions: 1.0.0, 1.2.0, 1.1.0 >Reporter: Brock Noland >Assignee: Brock Noland >Priority: Minor > Fix For: 2.0.0 > > Attachments: HIVE-11891.patch, HIVE-11891.patch, HIVE-11891.patch > > > At present it's extremely difficult to debug slow calls to the metastore. > Ideally there would be some basic means of doing so. -- This message was sent by Atlassian JIRA (v6.3.4#6332)