Github user DaveBirdsall commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1449#discussion_r168861011
--- Diff: core/sql/executor/ex_esp_frag_dir.cpp ---
@@ -101,11 +101,23 @@ ExEspFragInstanceDir::ExEspFragInstanceDir(CliGlobals
*cliGlobals,
pid_ = phandle.getPin();
tid_ = syscall(SYS_gettid);
- if (statsGlobals_ == NULL
- || (statsGlobals_ != NULL &&
- statsGlobals_->getVersion() !=
StatsGlobals::CURRENT_SHARED_OBJECTS_VERSION_))
+ NABoolean reportError = FALSE;
+ char msg[256];;
+ if (statsGlobals_ != NULL && pid_ >=
statsGlobals_->getConfiguredPidMax())
+ reportError = TRUE;
+ if ((statsGlobals_ == NULL)
+ || ((statsGlobals_ != NULL) &&
+ ((statsGlobals_->getVersion() !=
StatsGlobals::CURRENT_SHARED_OBJECTS_VERSION_) ||
+ (pid_ >= statsGlobals_->getConfiguredPidMax()))))
{
+ if (reportError) {
--- End diff --
Same comment as earlier.
---
