Ran
|
Jobs
1
|
Files
514
|
Run time
1min
|
Badge
Embed ▾
README BADGES
|
push
github
config: fix schema:get()/set() changes passed path This patch makes `schema:get()`, `schema:set()` and `config:get()` not change the passed path argument when it is a table. The problem was in the way the walkthrough process is set up during the `schema:get()` and `schema:set()` method execution. These methods used `walkthrough_start()` performing a shallow copy of the passed option table containing the information on the desired path. The value of the path was modified during the walkthrough itself. The patch makes the methods perform a shallow copy of the path field. It also fixes `config:get()` since it's internally uses `schema:get()`. This is how broken scenario could be reproduced when using `config:get()`: ``` tarantool> config = require('config') tarantool> path = {'fiber', 'slice'} tarantool> config:get(path) --- - err: 1 warn: 0.5 ... tarantool> path --- - [] ... ``` Closes #10855 NO_DOC=bugfix (cherry picked from commit fb5740cae)
68556 of 121788 branches covered (56.29%)
101171 of 115931 relevant lines covered (87.27%)
2668980.22 hits per line
Lines | Coverage | ∆ | File |
---|---|---|---|
1 |
95.36 |
-0.31% | src/box/raft.c |
1 |
92.12 |
0.05% | src/box/vinyl.c |
2 |
84.19 |
0.01% | src/box/xlog.c |
2 |
84.64 |
-0.33% | src/lib/core/say.c |
2 |
92.81 |
-1.44% | src/box/vy_point_lookup.c |
2 |
82.22 |
-4.44% | src/lib/core/latch.h |
2 |
90.75 |
-0.07% | src/box/box.cc |
3 |
87.47 |
-0.2% | src/box/vy_run.c |
4 |
90.55 |
-0.3% | src/box/applier.cc |
4 |
22.62 |
-0.08% | src/lib/uri/uri_parser.c |
9 |
88.14 |
-7.63% | src/box/vy_regulator.c |
ID | Job ID | Ran | Files | Coverage | |
---|---|---|---|---|---|
1 | 12182686276.1 | 514 |
87.27 |
GitHub Action Run |
Coverage | ∆ | File | Lines | Relevant | Covered | Missed | Hits/Line | Branch Hits | Branch Misses |
---|