Browse Source

HPCC-16859 Report failed to write sent query dll

Previously just a warning, which led to a misleading failed to
load dll error.

Signed-off-by: Jake Smith <jake.smith@lexisnexis.com>
Jake Smith 8 years ago
parent
commit
bd6bbf40de
1 changed files with 2 additions and 9 deletions
  1. 2 9
      thorlcr/slave/slavmain.cpp

+ 2 - 9
thorlcr/slave/slavmain.cpp

@@ -302,16 +302,9 @@ public:
                             }
                             catch (IException *e)
                             {
-                                StringBuffer msg("Failed to save dll, cwd = ");
-                                char buf[255];
-                                if (!GetCurrentDirectory(sizeof(buf), buf))
-                                {
-                                    ERRLOG("CJobListener::main: Current directory path too big, setting it to null");
-                                    buf[0] = 0;
-                                }
-                                msg.append(buf).append(", path = ").append(soPath);
-                                EXCLOG(e, msg.str());
+                                IException *e2 = ThorWrapException(e, "Failed to save dll: %s", soPath.str());
                                 e->Release();
+                                throw e2;
                             }
                             assertex(globals->getPropBool("Debug/@dllsToSlaves", true));
                             querySoCache.add(soPath.str());