Skip to content

Change headers format to str:json#319

Merged
fantix merged 5 commits intomasterfrom
str-headers
Jun 18, 2022
Merged

Change headers format to str:json#319
fantix merged 5 commits intomasterfrom
str-headers

Conversation

@fantix
Copy link
Member

@fantix fantix commented Jun 17, 2022

Headers now has the following format:

struct Header {
  // Header key.
  bytes           key;
  // Header data.
  bytes           value;
}

struct SomeMessage {
  ...
  // A set of message headers.
  uint16          num_headers;
  Header          headers[num_headers];
}
  • Header in ErrorMessage is not changed
  • ClientHandshake always send b'\0\0' (empty ext headers)
  • ServerHandshake no longer has ext headers

* Header in ErrorMessage is not changed
* ClientHandshake always send b'\0\0' (empty ext headers)
* ServerHandshake no longer has ext headers
@fantix fantix marked this pull request as ready for review June 17, 2022 21:35
@fantix fantix merged commit f17a4fd into master Jun 18, 2022
@fantix fantix deleted the str-headers branch June 18, 2022 15:02
elprans added a commit that referenced this pull request Jul 21, 2022
Changes
=======

* Remove DSN from `create_client()`
  (by @elprans in 9081105 for #303)

* Add support for protocol v1.0
  (by @fantix in 51344fe for #306)

* improve error when passing empty query arguments
  (by @nsidnev in 7a393ec)

* Accept args in execute() and use the new Execute message
  (by @fantix in 5298701 for #310)

* drop legacy agruments encoding with named tuple codec
  (by @nsidnev in ef96e76)

* Stop using Parse, replace use of headers with fields
  (by @elprans in b2c9e4b)

* Change headers format to str:json
  (by @fantix in f17a4fd for #319)

* Document rolling back a transaction
  (by @fmoor in 266f18b)

* Support EDGEDB_WAIT_UNTIL_AVAILABLE environment variable
  (by @fmoor in c27ab36)

* Add `with_globals()` and friends through state over the protocol
  (by @fantix in 8582ec1 for #315)

* Implement support for range types
  (by @elprans in a415b9f for #332)

* Implement support for cal::date_duration
  (by @elprans in e77615b for #335)

* Recover from failed COMMIT
  (by @fantix in 9f09e78)

* Allow <array<range<T>>> arguments
  (by @fmoor in 243250c for #351)
@elprans elprans mentioned this pull request Jul 21, 2022
fantix pushed a commit that referenced this pull request Jul 21, 2022
Changes
=======

* Remove DSN from `create_client()`
  (by @elprans in 9081105 for #303)

* Add support for protocol v1.0
  (by @fantix in 51344fe for #306)

* improve error when passing empty query arguments
  (by @nsidnev in 7a393ec)

* Accept args in execute() and use the new Execute message
  (by @fantix in 5298701 for #310)

* drop legacy agruments encoding with named tuple codec
  (by @nsidnev in ef96e76)

* Stop using Parse, replace use of headers with fields
  (by @elprans in b2c9e4b)

* Change headers format to str:json
  (by @fantix in f17a4fd for #319)

* Document rolling back a transaction
  (by @fmoor in 266f18b)

* Support EDGEDB_WAIT_UNTIL_AVAILABLE environment variable
  (by @fmoor in c27ab36)

* Add `with_globals()` and friends through state over the protocol
  (by @fantix in 8582ec1 for #315)

* Implement support for range types
  (by @elprans in a415b9f for #332)

* Implement support for cal::date_duration
  (by @elprans in e77615b for #335)

* Recover from failed COMMIT
  (by @fantix in 9f09e78)

* Allow <array<range<T>>> arguments
  (by @fmoor in 243250c for #351)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants