Paul Colomiets

Results 454 comments of Paul Colomiets

If the error is when migrations are being executed it means that database is initialized. You should be able to apply migration separately via `edgedb migrate` (or perhaps `edgedb instance...

I'm not sure why you want a factory in `--with-codec` and to export `JsonUnpackingCodec` as a factory rather than as a const, but other than that looks good. My thinking...

The alternative would be to set the whole codec collection: ```python connection.with_codecs(edgedb.codecs.DEFAULT) connection.with_codecs(edgedb.codecs.DEFAULT_WITH_UNPACKED_JSON) ``` The collection itself has interface like this: ```python class Codecs: def get_type_name(self, descriptor) -> str: #...

To simplify single type case, we can also probably allow both ways: ```python connection.with_codec(MyDecimalCodec) # being equivalent to connection.with_codecs( connection.get_codecs().with_codec(MyDecimalCodec) ) ``` But I think this means copying the whole...

You cannot start/stop linked instances, as there is no generic way to do that. You should use docker commands for start/stop if you have started the server using docker. I...

There is no such thing out of the box. We can probably add one, but I guess there are few more complications: 1. Some hop-by-hop things might need to be...

Hi! Sorry, for a late response. Please open a PR! It's easier to judge looking at the code.

I believe it may look like: ``` try: base_class = InterrutedError except NameError: base_class = OSError class NNInterruptedError(BaseNNError, base_class): pass ``` This way it would work in both python versions

Note to myself: also check that `project init` do the same check.

We don't have such field for now. But also our record types are bad for caches in the following ways: 1. Non-static lifetimes 2. Records should be order-independent. E.g. `127.0.0.1,...