[TRAFODION-3007] update statistics report error on volatile table #1492
Conversation
|
Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2509/ |
|
Test Failed. https://jenkins.esgyn.com/job/Check-PR-master/2509/ |
|
New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2514/ |
|
Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/2514/ |
DaveBirdsall
left a comment
DaveBirdsall
left a comment
There was a problem hiding this comment.
+1 Looks good. One thing to check in your unit testing: Please be sure that any SB_HISTOGRAM* table in the volatile schema gets dropped when the schema gets dropped.
core/sql/ustat/hs_parser.cpp
Outdated
| HSHandleError(retcode); // causes a return from this function | ||
|
|
||
| // The code below is old code that will be needed once we turn on | ||
| // support for UPDATE STATISTICS on volatile tables. We leave it here |
There was a problem hiding this comment.
We could delete the comments "The code below is old code..." since we are turning support back on.
There was a problem hiding this comment.
Thanks Dave. SB_HISTOGRAM* table in the volatile schema gets dropped when the schema gets dropped. I had tested that.
…emove useless comments
|
New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2516/ |
|
Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/2516/ |
4 participants
No description provided.