citusdata / citusdata/test-automation

The syntax for pg.set_config is awkward

Open
#34 0 comments 0 reactions 0 assignees View on GitHub
usability
Dominant language
C
Stars
15
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Because of the way fabric's parsing works, if you want to include a comma you must escape it with a backslash, like so:

```
# this works
fab pg.set_config:shared_preload_libraries,'citus\,cstore_fdw'
# you have to escape it even if you use keyword args (the below does not work)
fab pg.set_config:shared_preload_libraries='citus\,cstore_fdw'
```

We could monkeypatch the parsing to make it something more reasonable.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.