- 05 5月, 2020 1 次提交
-
-
由 yifan hao 提交于
This patch refactors the initialization and cleanup path of dnode with a table driven method. This fixes the following issues: 1. Before the patch, if dnodeInitRead() fails, the cleanup path also runs dnodeCleanupWrite(), which will free wWorkerPool.writeWorker that's never allocated. (The code before this patch will not crash though, because wWorkerPool is zero-initialized global variable and therefore the accidental free will be a nop). 2. In general the order of calling cleanup function should be reverse to the order of calling init function, but this is not the case prior to this patch (see dnodeCleanupMnode() and dnodeCleanupMgmt()). * Bonus fix This patch also fixes a missing free for readPool.readWorker. * Testing I plan to run the test script ./test-all.sh, but was not able to do so. Is there a reference somewhere I can look up?
-
- 04 5月, 2020 12 次提交
-
-
由 slguan 提交于
-
由 slguan 提交于
Feature/2.0tsdb
-
由 hzcheng 提交于
-
由 slguan 提交于
Proper error handling in dnodeOpenVnodes / dnodeCloseVnodes
-
由 slguan 提交于
[Trivial] Fix initialization code of TSDB_MOD_MGMT module.
-
由 slguan 提交于
rpc shall not check peer IP address, it shall only use FQDN
-
由 jtao1735 提交于
-
由 jtao1735 提交于
-
由 jtao1735 提交于
-
由 yifan hao 提交于
* Description These two functions do not properly handle any error returned from dnodeGetVnodeList(). This patch als changes the signature of dnodeGetVnodeList() to make sure it returns only status, and number of vnodes is passed out through a separate argument. * Testing Before the patch, if I start taosd and then remove 'vnode' folder, the shutdown message would include the following: 05/03 14:26:17.158204 0x140025950476096 DND total vnodes:-2147483296 are all closed This shows the code uses error code as vnode count. After the patch, it outputs proper error message: 05/03 14:30:45.044762 0x139933723604800 DND Get dnode list failed
-
由 yifan hao 提交于
* Description These two functions do not properly handle any error returned from dnodeGetVnodeList(). This patch als changes the signature of dnodeGetVnodeList() to make sure it returns only status, and number of vnodes is passed out through a separate argument. * Testing Before the patch, if I start taosd and then remove 'vnode' folder, the shutdown message would include the following: 05/03 14:26:17.158204 0x140025950476096 DND total vnodes:-2147483296 are all closed This shows the code uses error code as vnode count. After the patch, it outputs proper error message: 05/03 14:30:45.044762 0x139933723604800 DND Get dnode list failed
-
由 yifan hao 提交于
* Description This patch fixes enablement of TSDB_MOD_MGMT module, which was incorrectly assign to 'name' variable. * Testing Compile the code. Feel free to suggest more testing that needs to be done. Given that 'tsModule' is zero initialized, I don't think there's any functional difference for this patch.
-
- 03 5月, 2020 13 次提交
-
-
由 slguan 提交于
-
由 slguan 提交于
if resposne is received, and no corresponding connection there, it sh…
-
由 jtao1735 提交于
-
由 hzcheng 提交于
-
由 hzcheng 提交于
-
由 hzcheng 提交于
-
由 haojun Liao 提交于
maxId is from 1 to maxSession
-
由 jtao1735 提交于
-
由 slguan 提交于
-
由 slguan 提交于
-
由 slguan 提交于
-
由 slguan 提交于
initialize SCqCfg structure
-
由 jtao1735 提交于
remove two unsed fields in tsclient.h
-
- 02 5月, 2020 14 次提交