• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

taosdata / TDengine / #3545

02 Dec 2024 06:22AM UTC coverage: 60.839% (-0.04%) from 60.88%
#3545

push

travis-ci

web-flow
Merge pull request #28961 from taosdata/fix/refactor-vnode-management-open-vnode

fix/refactor-vnode-management-open-vnode

120592 of 253473 branches covered (47.58%)

Branch coverage included in aggregate %.

102 of 145 new or added lines in 3 files covered. (70.34%)

477 existing lines in 108 files now uncovered.

201840 of 276506 relevant lines covered (73.0%)

19392204.25 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

66.89
/source/dnode/mgmt/mgmt_vnode/src/vmFile.c
1
/*
2
 * Copyright (c) 2019 TAOS Data, Inc. <jhtao@taosdata.com>
3
 *
4
 * This program is free software: you can use, redistribute, and/or modify
5
 * it under the terms of the GNU Affero General Public License, version 3
6
 * or later ("AGPL"), as published by the Free Software Foundation.
7
 *
8
 * This program is distributed in the hope that it will be useful, but WITHOUT
9
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
10
 * FITNESS FOR A PARTICULAR PURPOSE.
11
 *
12
 * You should have received a copy of the GNU Affero General Public License
13
 * along with this program. If not, see <http:www.gnu.org/licenses/>.
14
 */
15

16
#define _DEFAULT_SOURCE
17
#include "tjson.h"
18
#include "vmInt.h"
19

20
#define MAX_CONTENT_LEN 2 * 1024 * 1024
21

22
int32_t vmGetAllVnodeListFromHash(SVnodeMgmt *pMgmt, int32_t *numOfVnodes, SVnodeObj ***ppVnodes) {
21,730✔
23
  (void)taosThreadRwlockRdlock(&pMgmt->lock);
21,730✔
24

25
  int32_t num = 0;
21,730✔
26
  int32_t size = taosHashGetSize(pMgmt->runngingHash);
21,730✔
27
  int32_t closedSize = taosHashGetSize(pMgmt->closedHash);
21,730✔
28
  size += closedSize;
21,730✔
29
  SVnodeObj **pVnodes = taosMemoryCalloc(size, sizeof(SVnodeObj *));
21,730✔
30
  if (pVnodes == NULL) {
21,730!
31
    (void)taosThreadRwlockUnlock(&pMgmt->lock);
×
32
    return terrno;
×
33
  }
34

35
  void *pIter = taosHashIterate(pMgmt->runngingHash, NULL);
21,730✔
36
  while (pIter) {
137,623✔
37
    SVnodeObj **ppVnode = pIter;
115,893✔
38
    SVnodeObj  *pVnode = *ppVnode;
115,893✔
39
    if (pVnode && num < size) {
231,786!
40
      int32_t refCount = atomic_add_fetch_32(&pVnode->refCount, 1);
115,893✔
41
      dTrace("vgId:%d,acquire vnode, vnode:%p, ref:%d", pVnode->vgId, pVnode, refCount);
115,893✔
42
      pVnodes[num++] = (*ppVnode);
115,893✔
43
      pIter = taosHashIterate(pMgmt->runngingHash, pIter);
115,893✔
44
    } else {
NEW
45
      taosHashCancelIterate(pMgmt->runngingHash, pIter);
×
46
    }
47
  }
48

49
  pIter = taosHashIterate(pMgmt->closedHash, NULL);
21,730✔
50
  while (pIter) {
21,730!
51
    SVnodeObj **ppVnode = pIter;
×
52
    SVnodeObj  *pVnode = *ppVnode;
×
53
    if (pVnode && num < size) {
×
54
      int32_t refCount = atomic_add_fetch_32(&pVnode->refCount, 1);
×
55
      dTrace("vgId:%d, acquire vnode, vnode:%p, ref:%d", pVnode->vgId, pVnode, refCount);
×
56
      pVnodes[num++] = (*ppVnode);
×
57
      pIter = taosHashIterate(pMgmt->closedHash, pIter);
×
58
    } else {
59
      taosHashCancelIterate(pMgmt->closedHash, pIter);
×
60
    }
61
  }
62

63
  (void)taosThreadRwlockUnlock(&pMgmt->lock);
21,730✔
64
  *numOfVnodes = num;
21,730✔
65
  *ppVnodes = pVnodes;
21,730✔
66

67
  return 0;
21,730✔
68
}
69

70
int32_t vmGetAllVnodeListFromHashWithCreating(SVnodeMgmt *pMgmt, int32_t *numOfVnodes, SVnodeObj ***ppVnodes) {
11,186✔
71
  (void)taosThreadRwlockRdlock(&pMgmt->lock);
11,186✔
72

73
  int32_t num = 0;
11,186✔
74
  int32_t size = taosHashGetSize(pMgmt->runngingHash);
11,186✔
75
  int32_t creatingSize = taosHashGetSize(pMgmt->creatingHash);
11,186✔
76
  size += creatingSize;
11,186✔
77
  SVnodeObj **pVnodes = taosMemoryCalloc(size, sizeof(SVnodeObj *));
11,186✔
78
  if (pVnodes == NULL) {
11,186!
79
    (void)taosThreadRwlockUnlock(&pMgmt->lock);
×
80
    return terrno;
×
81
  }
82

83
  void *pIter = taosHashIterate(pMgmt->runngingHash, NULL);
11,186✔
84
  while (pIter) {
44,102✔
85
    SVnodeObj **ppVnode = pIter;
32,916✔
86
    SVnodeObj  *pVnode = *ppVnode;
32,916✔
87
    if (pVnode && num < size) {
65,832!
88
      int32_t refCount = atomic_add_fetch_32(&pVnode->refCount, 1);
32,916✔
89
      dTrace("vgId:%d,acquire vnode, vnode:%p, ref:%d", pVnode->vgId, pVnode, refCount);
32,916✔
90
      pVnodes[num++] = (*ppVnode);
32,916✔
91
      pIter = taosHashIterate(pMgmt->runngingHash, pIter);
32,916✔
92
    } else {
NEW
93
      taosHashCancelIterate(pMgmt->runngingHash, pIter);
×
94
    }
95
  }
96

97
  pIter = taosHashIterate(pMgmt->creatingHash, NULL);
11,186✔
98
  while (pIter) {
21,799✔
99
    SVnodeObj **ppVnode = pIter;
10,613✔
100
    SVnodeObj  *pVnode = *ppVnode;
10,613✔
101
    if (pVnode && num < size) {
21,226!
102
      int32_t refCount = atomic_add_fetch_32(&pVnode->refCount, 1);
10,613✔
103
      dTrace("vgId:%d, acquire vnode, vnode:%p, ref:%d", pVnode->vgId, pVnode, refCount);
10,613✔
104
      pVnodes[num++] = (*ppVnode);
10,613✔
105
      pIter = taosHashIterate(pMgmt->creatingHash, pIter);
10,613✔
106
    } else {
107
      taosHashCancelIterate(pMgmt->creatingHash, pIter);
×
108
    }
109
  }
110
  (void)taosThreadRwlockUnlock(&pMgmt->lock);
11,186✔
111

112
  *numOfVnodes = num;
11,186✔
113
  *ppVnodes = pVnodes;
11,186✔
114

115
  return 0;
11,186✔
116
}
117

118
int32_t vmGetVnodeListFromHash(SVnodeMgmt *pMgmt, int32_t *numOfVnodes, SVnodeObj ***ppVnodes) {
7,516✔
119
  (void)taosThreadRwlockRdlock(&pMgmt->lock);
7,516✔
120

121
  int32_t     num = 0;
7,516✔
122
  int32_t     size = taosHashGetSize(pMgmt->runngingHash);
7,516✔
123
  SVnodeObj **pVnodes = taosMemoryCalloc(size, sizeof(SVnodeObj *));
7,516✔
124
  if (pVnodes == NULL) {
7,516!
125
    (void)taosThreadRwlockUnlock(&pMgmt->lock);
×
126
    return terrno;
×
127
  }
128

129
  void *pIter = taosHashIterate(pMgmt->runngingHash, NULL);
7,516✔
130
  while (pIter) {
46,671✔
131
    SVnodeObj **ppVnode = pIter;
39,155✔
132
    SVnodeObj  *pVnode = *ppVnode;
39,155✔
133
    if (pVnode && num < size) {
78,310!
134
      int32_t refCount = atomic_add_fetch_32(&pVnode->refCount, 1);
39,155✔
135
      dTrace("vgId:%d, acquire vnode, vnode:%p, ref:%d", pVnode->vgId, pVnode, refCount);
39,155✔
136
      pVnodes[num++] = (*ppVnode);
39,155✔
137
      pIter = taosHashIterate(pMgmt->runngingHash, pIter);
39,155✔
138
    } else {
NEW
139
      taosHashCancelIterate(pMgmt->runngingHash, pIter);
×
140
    }
141
  }
142

143
  (void)taosThreadRwlockUnlock(&pMgmt->lock);
7,516✔
144
  *numOfVnodes = num;
7,516✔
145
  *ppVnodes = pVnodes;
7,516✔
146

147
  return 0;
7,516✔
148
}
149

150
static int32_t vmDecodeVnodeList(SJson *pJson, SVnodeMgmt *pMgmt, SWrapperCfg **ppCfgs, int32_t *numOfVnodes) {
407✔
151
  int32_t      code = -1;
407✔
152
  SWrapperCfg *pCfgs = NULL;
407✔
153
  *ppCfgs = NULL;
407✔
154

155
  SJson *vnodes = tjsonGetObjectItem(pJson, "vnodes");
407✔
156
  if (vnodes == NULL) return TSDB_CODE_INVALID_JSON_FORMAT;
407!
157

158
  int32_t vnodesNum = cJSON_GetArraySize(vnodes);
407✔
159
  if (vnodesNum > 0) {
407✔
160
    pCfgs = taosMemoryCalloc(vnodesNum, sizeof(SWrapperCfg));
405✔
161
    if (pCfgs == NULL) return terrno;
405!
162
  }
163

164
  for (int32_t i = 0; i < vnodesNum; ++i) {
1,442✔
165
    SJson *vnode = tjsonGetArrayItem(vnodes, i);
1,035✔
166
    if (vnode == NULL) {
1,035!
167
      code = TSDB_CODE_INVALID_JSON_FORMAT;
×
168
      goto _OVER;
×
169
    }
170

171
    SWrapperCfg *pCfg = &pCfgs[i];
1,035✔
172
    tjsonGetInt32ValueFromDouble(vnode, "vgId", pCfg->vgId, code);
1,035✔
173
    if (code != 0) goto _OVER;
1,035!
174
    tjsonGetInt32ValueFromDouble(vnode, "dropped", pCfg->dropped, code);
1,035✔
175
    if (code != 0) goto _OVER;
1,035!
176
    tjsonGetInt32ValueFromDouble(vnode, "vgVersion", pCfg->vgVersion, code);
1,035✔
177
    if (code != 0) goto _OVER;
1,035!
178
    tjsonGetInt32ValueFromDouble(vnode, "diskPrimary", pCfg->diskPrimary, code);
1,035✔
179
    if (code != 0) goto _OVER;
1,035!
180
    tjsonGetInt32ValueFromDouble(vnode, "toVgId", pCfg->toVgId, code);
1,035✔
181
    if (code != 0) goto _OVER;
1,035!
182

183
    snprintf(pCfg->path, sizeof(pCfg->path), "%s%svnode%d", pMgmt->path, TD_DIRSEP, pCfg->vgId);
1,035✔
184
  }
185

186
  code = 0;
407✔
187
  *ppCfgs = pCfgs;
407✔
188
  *numOfVnodes = vnodesNum;
407✔
189

190
_OVER:
407✔
191
  if (*ppCfgs == NULL) taosMemoryFree(pCfgs);
407✔
192
  return code;
407✔
193
}
194

195
int32_t vmGetVnodeListFromFile(SVnodeMgmt *pMgmt, SWrapperCfg **ppCfgs, int32_t *numOfVnodes) {
2,412✔
196
  int32_t      code = -1;
2,412✔
197
  TdFilePtr    pFile = NULL;
2,412✔
198
  char        *pData = NULL;
2,412✔
199
  SJson       *pJson = NULL;
2,412✔
200
  char         file[PATH_MAX] = {0};
2,412✔
201
  SWrapperCfg *pCfgs = NULL;
2,412✔
202
  snprintf(file, sizeof(file), "%s%svnodes.json", pMgmt->path, TD_DIRSEP);
2,412✔
203

204
  if (taosStatFile(file, NULL, NULL, NULL) < 0) {
2,412✔
205
    code = terrno;
2,005✔
206
    dInfo("vnode file:%s not exist, reason:%s", file, tstrerror(code));
2,005!
207
    code = 0;
2,005✔
208
    return code;
2,005✔
209
  }
210

211
  pFile = taosOpenFile(file, TD_FILE_READ);
407✔
212
  if (pFile == NULL) {
407!
213
    code = terrno;
×
214
    dError("failed to open vnode file:%s since %s", file, tstrerror(code));
×
215
    goto _OVER;
×
216
  }
217

218
  int64_t size = 0;
407✔
219
  code = taosFStatFile(pFile, &size, NULL);
407✔
220
  if (code != 0) {
407!
221
    dError("failed to fstat mnode file:%s since %s", file, tstrerror(code));
×
222
    goto _OVER;
×
223
  }
224

225
  pData = taosMemoryMalloc(size + 1);
407✔
226
  if (pData == NULL) {
407!
227
    code = terrno;
×
228
    goto _OVER;
×
229
  }
230

231
  if (taosReadFile(pFile, pData, size) != size) {
407!
232
    code = terrno;
×
233
    dError("failed to read vnode file:%s since %s", file, tstrerror(code));
×
234
    goto _OVER;
×
235
  }
236

237
  pData[size] = '\0';
407✔
238

239
  pJson = tjsonParse(pData);
407✔
240
  if (pJson == NULL) {
407!
241
    code = TSDB_CODE_INVALID_JSON_FORMAT;
×
242
    goto _OVER;
×
243
  }
244

245
  if (vmDecodeVnodeList(pJson, pMgmt, ppCfgs, numOfVnodes) < 0) {
407!
246
    code = TSDB_CODE_INVALID_JSON_FORMAT;
×
247
    goto _OVER;
×
248
  }
249

250
  code = 0;
407✔
251
  dInfo("succceed to read vnode file %s", file);
407!
252

253
_OVER:
×
254
  if (pData != NULL) taosMemoryFree(pData);
407!
255
  if (pJson != NULL) cJSON_Delete(pJson);
407!
256
  if (pFile != NULL) taosCloseFile(&pFile);
407!
257

258
  if (code != 0) {
407!
259
    dError("failed to read vnode file:%s since %s", file, tstrerror(code));
×
260
  }
261
  return code;
407✔
262
}
263

264
static int32_t vmEncodeVnodeList(SJson *pJson, SVnodeObj **ppVnodes, int32_t numOfVnodes) {
21,730✔
265
  int32_t code = 0;
21,730✔
266
  SJson  *vnodes = tjsonCreateArray();
21,730✔
267
  if (vnodes == NULL) {
21,730!
268
    return terrno;
×
269
  }
270
  if ((code = tjsonAddItemToObject(pJson, "vnodes", vnodes)) < 0) {
21,730!
271
    tjsonDelete(vnodes);
×
272
    return code;
×
273
  };
274

275
  for (int32_t i = 0; i < numOfVnodes; ++i) {
137,614✔
276
    SVnodeObj *pVnode = ppVnodes[i];
115,882✔
277
    if (pVnode == NULL) continue;
115,882!
278

279
    SJson *vnode = tjsonCreateObject();
115,882✔
280
    if (vnode == NULL) return terrno;
115,888!
281
    if ((code = tjsonAddDoubleToObject(vnode, "vgId", pVnode->vgId)) < 0) return code;
115,888!
282
    if ((code = tjsonAddDoubleToObject(vnode, "dropped", pVnode->dropped)) < 0) return code;
115,887!
283
    if ((code = tjsonAddDoubleToObject(vnode, "vgVersion", pVnode->vgVersion)) < 0) return code;
115,887!
284
    if ((code = tjsonAddDoubleToObject(vnode, "diskPrimary", pVnode->diskPrimary)) < 0) return code;
115,888!
285
    if (pVnode->toVgId) {
115,886✔
286
      if ((code = tjsonAddDoubleToObject(vnode, "toVgId", pVnode->toVgId)) < 0) return code;
88!
287
    }
288
    if ((code = tjsonAddItemToArray(vnodes, vnode)) < 0) return code;
115,886!
289
  }
290

291
  return 0;
21,732✔
292
}
293

294
int32_t vmWriteVnodeListToFile(SVnodeMgmt *pMgmt) {
21,730✔
295
  int32_t     code = -1;
21,730✔
296
  char       *buffer = NULL;
21,730✔
297
  SJson      *pJson = NULL;
21,730✔
298
  TdFilePtr   pFile = NULL;
21,730✔
299
  SVnodeObj **ppVnodes = NULL;
21,730✔
300
  char        file[PATH_MAX] = {0};
21,730✔
301
  char        realfile[PATH_MAX] = {0};
21,730✔
302
  int32_t     lino = 0;
21,730✔
303
  int32_t     ret = -1;
21,730✔
304

305
  int32_t nBytes = snprintf(file, sizeof(file), "%s%svnodes_tmp.json", pMgmt->path, TD_DIRSEP);
21,730✔
306
  if (nBytes <= 0 || nBytes >= sizeof(file)) {
21,730!
307
    return TSDB_CODE_OUT_OF_RANGE;
×
308
  }
309

310
  nBytes = snprintf(realfile, sizeof(realfile), "%s%svnodes.json", pMgmt->path, TD_DIRSEP);
21,730✔
311
  if (nBytes <= 0 || nBytes >= sizeof(realfile)) {
21,730!
312
    return TSDB_CODE_OUT_OF_RANGE;
×
313
  }
314

315
  int32_t numOfVnodes = 0;
21,730✔
316
  TAOS_CHECK_GOTO(vmGetAllVnodeListFromHash(pMgmt, &numOfVnodes, &ppVnodes), &lino, _OVER);
21,730!
317

318
  // terrno = TSDB_CODE_OUT_OF_MEMORY;
319
  pJson = tjsonCreateObject();
21,730✔
320
  if (pJson == NULL) {
21,730!
321
    code = terrno;
×
322
    goto _OVER;
×
323
  }
324
  TAOS_CHECK_GOTO(vmEncodeVnodeList(pJson, ppVnodes, numOfVnodes), &lino, _OVER);
21,730!
325

326
  buffer = tjsonToString(pJson);
21,730✔
327
  if (buffer == NULL) {
21,730!
328
    code = TSDB_CODE_INVALID_JSON_FORMAT;
×
329
    lino = __LINE__;
×
330
    goto _OVER;
×
331
  }
332

333
  code = taosThreadMutexLock(&pMgmt->fileLock);
21,730✔
334
  if (code != 0) {
21,730!
335
    lino = __LINE__;
×
336
    goto _OVER;
×
337
  }
338

339
  pFile = taosOpenFile(file, TD_FILE_CREATE | TD_FILE_WRITE | TD_FILE_TRUNC | TD_FILE_WRITE_THROUGH);
21,730✔
340
  if (pFile == NULL) {
21,730!
341
    code = terrno;
×
342
    lino = __LINE__;
×
343
    goto _OVER1;
×
344
  }
345

346
  int32_t len = strlen(buffer);
21,730✔
347
  if (taosWriteFile(pFile, buffer, len) <= 0) {
21,730!
348
    code = terrno;
×
349
    lino = __LINE__;
×
350
    goto _OVER1;
×
351
  }
352
  if (taosFsyncFile(pFile) < 0) {
21,730!
353
    code = TAOS_SYSTEM_ERROR(errno);
×
354
    lino = __LINE__;
×
355
    goto _OVER1;
×
356
  }
357

358
  code = taosCloseFile(&pFile);
21,730✔
359
  if (code != 0) {
21,730!
360
    code = TAOS_SYSTEM_ERROR(errno);
×
361
    lino = __LINE__;
×
362
    goto _OVER1;
×
363
  }
364
  TAOS_CHECK_GOTO(taosRenameFile(file, realfile), &lino, _OVER1);
21,730!
365

366
  dInfo("succeed to write vnodes file:%s, vnodes:%d", realfile, numOfVnodes);
21,730!
367

368
_OVER1:
×
369
  ret = taosThreadMutexUnlock(&pMgmt->fileLock);
21,730✔
370
  if (ret != 0) {
21,730!
371
    dError("failed to unlock since %s", tstrerror(ret));
×
372
  }
373

374
_OVER:
21,730✔
375
  if (pJson != NULL) tjsonDelete(pJson);
21,730!
376
  if (buffer != NULL) taosMemoryFree(buffer);
21,730!
377
  if (pFile != NULL) taosCloseFile(&pFile);
21,730!
378
  if (ppVnodes != NULL) {
21,730!
379
    for (int32_t i = 0; i < numOfVnodes; ++i) {
137,623✔
380
      SVnodeObj *pVnode = ppVnodes[i];
115,893✔
381
      if (pVnode != NULL) {
115,893!
382
        vmReleaseVnode(pMgmt, pVnode);
115,893✔
383
      }
384
    }
385
    taosMemoryFree(ppVnodes);
21,730✔
386
  }
387

388
  if (code != 0) {
21,730!
389
    dError("failed to write vnodes file:%s at line:%d since %s, vnodes:%d", realfile, lino, tstrerror(code),
×
390
           numOfVnodes);
391
  }
392
  return code;
21,730✔
393
}
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc