Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

qvalue: UUID array type #2327

Merged
merged 6 commits into from
Dec 6, 2024
Merged

qvalue: UUID array type #2327

merged 6 commits into from
Dec 6, 2024

Conversation

serprex
Copy link
Contributor

@serprex serprex commented Dec 6, 2024

testing ci for #2326

@serprex serprex changed the title Test UUID array type qvalue: UUID array type Dec 6, 2024
@serprex serprex merged commit 572aa1d into main Dec 6, 2024
9 checks passed
@serprex serprex deleted the test-uuid-array-type branch December 6, 2024 16:28
iskakaushik pushed a commit that referenced this pull request Dec 16, 2024
Building on top of #2327 there are some missing pieces required to sync
this data into BQ

Have tested these changes locally

![image](https://github.com/user-attachments/assets/81d97117-91f7-40c9-8cfc-5c7b5c65a35f)


Using this setup
```sh
#!/usr/bin/env bash
set -xeuo pipefail

# This script creates databases on the PeerDB internal cluster to be used as peers later.

CONNECTION_STRING="${1:-postgres://postgres:postgres@localhost:9901/postgres}"

if ! type psql >/dev/null 2>&1; then
  echo "psql not found on PATH, exiting"
  exit 1
fi

psql "$CONNECTION_STRING" << EOF

--- Create the databases
DROP DATABASE IF EXISTS source;
CREATE DATABASE source;
DROP DATABASE IF EXISTS target;
CREATE DATABASE target;

--- Switch to source database
\c source

CREATE EXTENSION IF NOT EXISTS "uuid-ossp";

--- Create the source table
DROP TABLE IF EXISTS source CASCADE;
CREATE TABLE source (
    id uuid DEFAULT uuid_generate_v4(),
    related_ids uuid[],
    PRIMARY KEY (id)
);

CREATE PUBLICATION source_publication FOR TABLE source;

-- insert mock rows into source with valid uuid values
-- INSERT INTO source (related_ids) VALUES (ARRAY[uuid_generate_v4(), uuid_generate_v4()]);

-- Switch to target database
\c target

EOF

```
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.

3 participants