Allow to traverse over crud.cfg using pairs()
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
Research direction
Start by reading issue #244 and the referenced discussion in pull request #244, then review the related thoughts in Tarantool issue #4521. The issue presents several possible APIs for traversing or copying crud.cfg without selecting one, so completion requires an agreed design and corresponding documentation or behavior.
Written by the indexing model from the issue text.
Description
Follows up #244
Unlike
pairs(box.cfg),pairs(crud.cfg)will now work. Not really a bit deal, since here we have no backward compatibility guarantees.However maybe it worth to add
crud.cfg:pairs()and mention it in the documentation: just to don't let users write a wrong code withpairs(crud.cfg)orfun.iter(crud.cfg). Or addcrud.cfg:unwrap(), which returns a pure table. Orcrud.cfg:copy(), which returns a copy. Don't know.Maybe it is not needed for anyone. OTOH, using
fun.chain()to merge several configs is quite useful. Say, when you have a default one, one collected from env and one provided in arguments. But whether we can need a current config is such chains? Don't know.Let's ignore it for now. It is not critical.
I also added some more thoughts into https://github.com/tarantool/tarantool/issues/4521, if you're interested.
Originally posted by @Totktonada in https://github.com/tarantool/crud/pull/244#discussion_r811481219
- Dominant language
- Lua
- Stars
- 43
- Forks
- 17
- Avg merge
- 28m
- Merged PRs (30d)
- 2
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from tarantool/crud
-
Difficulty 3/5 1-2 days Newbie friendliness 72/100
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
-
Difficulty 3/5 1-2 days Newbie friendliness 68/100
-
performance
Difficulty 2/5 1-3 hours Newbie friendliness 55/100
-
Difficulty 3/5 1-2 days Newbie friendliness 38/100
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
-
[Eco] Egg Config Open
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
pterodactyl/game-eggs#634 ·
-
Feature Request
Difficulty 1/5 Under an hour Newbie friendliness 72/100
Questie/QuestieTrace#33 ·