Previously when running `pico.sql [[ create table foo ]]` we would get
this output:
```
---
- null
- "sbroad: rule parsing error:  --> 1:9\n  |\n1 |  create table foo \n  |  ^---\n
  \ |\n  = expected Unique"
...
```

But now the output will be:
```
---
- null
- |+
  sbroad: rule parsing error:  --> 1:9
    |
  1 |  create table foo
    |         ^---
    |
    = expected Unique

...
```

These tests are quire lengthy and are useful only when we changed the
patchset.

So run them always on master, and if `certification_patches` dir
contains any changes. Otherwise use `when: manual`

this fixes the following bug:
```
picodata run --data-dir tmp/i1/
C> invalid socket path: tmp/i1//admin.sock
[supervisor:214288] no ipc message from child
[supervisor:214288] subprocess 214289 exited with code 1
```

We don't need to replace $n with question marks any more, as we support
the pg parameters format.

See https://git.picodata.io/picodata/picodata/sbroad/-/merge_requests/377
and https://git.picodata.io/picodata/picodata/sbroad/-/issues/543.

Виталий Шунков authored 9 months ago and Dmitry Rodionov committed 9 months ago

https://git.picodata.io/picodata/picodata/picodata/-/work_items/563

Close #635

Reasons:
- it's not safe to split raft entries as atomicness is lost
- the original feature was not tested

Dmitry Ivanov authored 9 months ago and Dmitry Ivanov committed 9 months ago

This reverts commit 95807a30.

Dmitry Ivanov authored 9 months ago and Dmitry Ivanov committed 9 months ago

This reverts commit 9f8687cd.

Dmitry Ivanov authored 9 months ago and Dmitry Ivanov committed 9 months ago

Previously, we took advantage of a faster psycopg-binary, but this
dependency tends to fall short whenever we produce a bad server
packet, leading to obscure backtraces.

Maksim Kaitmazian authored 11 months ago and Dmitry Ivanov committed 9 months ago

pgproto used to interact with picodata through lua entrypoins, but after
making pgproto a picodata module, we can get rid of this lua layer and
simply call corresponding rust functions.

Maksim Kaitmazian authored 11 months ago and Dmitry Ivanov committed 9 months ago

After moving pgproto into picodata we can merge their implementations of
Describe into a singe module.

Maksim Kaitmazian authored 11 months ago and Dmitry Ivanov committed 9 months ago

The backend module implements the logic for handling postgres messages.
Note that this commit doesn't bring anything new: it just moves picodata's
handlers into pgproto.

See: https://git.picodata.io/picodata/picodata/picodata/-/issues/630