DaveBirdsall commented on a change in pull request #1821: [TRAFODION-3291] Fix
core when multi-column stats are done on lots of columns
URL: https://github.com/apache/trafodion/pull/1821#discussion_r272375471
##########
File path: core/sql/ustat/hs_log.h
##########
@@ -277,5 +283,8 @@ class HSLogMan
timeval prevTime_;
char title_[MAX_TIMING_EVENTS][1000];
static THREAD_P HSLogMan* instance_; /* 1 and only 1 instance
*/
+
+ // intended as a long-term replacement for msg + Log()
+ NAString logStreamBuffer_;
Review comment:
Thanks for the review. I'll explore putting logStreamBuffer_ on an NAHeap in
a later pull request. One bit of good news is that this code runs in the
tdm_arkcmp process, so we won't be affecting global heap in mxosrvr.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
