sql: fix a couple of spots where we missed JSONPATH and LTREE#156691
sql: fix a couple of spots where we missed JSONPATH and LTREE#156691craig[bot] merged 1 commit intocockroachdb:masterfrom
Conversation
|
This change is |
We forgot to include `DJsonpath` and `DLTree` into some of the type switches when converting datums into string format for miscellaneous builtins. This is now fixed. I decided to omit a release note given these seem like edge cases. Release note: None
3b361b0 to
8de38b7
Compare
|
TFTR! bors r+ |
156691: sql: fix a couple of spots where we missed JSONPATH and LTREE r=yuzefovich a=yuzefovich We forgot to include `DJsonpath` and `DLTree` into some of the type switches when converting datums into string format for miscellaneous builtins. This is now fixed. I decided to omit a release note given these seem like edge cases. Fixes: #156511. Fixes: #156611. Release note: None 156954: roachtest/npgsql: skip on s390x r=fqazi a=fqazi Previously, the npgsql test was running on S390X, which doesn't have the required binaries. To address this, this patch excludes the IBM cloud. Fixes: #156622 Release note: None 156958: ci: make env vars across cloud providers consistent for roachtest r=srosenberg,rickystewart a=williamchoe3 Noticed some inconsistency, this aligns gce azure aws ibm Just with `EXTRA_ROACHTEST_ARGS` and `ALWAYS_COLLECT_ARTIFACTS` Co-authored-by: Yahor Yuzefovich <yahor@cockroachlabs.com> Co-authored-by: Faizan Qazi <faizan@cockroachlabs.com> Co-authored-by: William Choe <williamchoe3@gmail.com>
|
Build failed (retrying...): |
|
Based on the specified backports for this PR, I applied new labels to the following linked issue(s). Please adjust the labels as needed to match the branches actually affected by the issue(s), including adding any known older branches. Issue #156511: branch-release-25.3, branch-release-25.4. Issue #156611: branch-release-25.3. 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
|
Encountered an error creating backports. Some common things that can go wrong:
You might need to create your backport manually using the backport tool. error creating merge commit from 8de38b7 to blathers/backport-release-25.3-156691: POST https://api.github.com/repos/yuzefovich/cockroach/merges: 409 Merge conflict [] you may need to manually resolve merge conflicts with the backport tool. Backport to branch 25.3.x failed. See errors above. 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
We forgot to include
DJsonpathandDLTreeinto some of the type switches when converting datums into string format for miscellaneous builtins. This is now fixed. I decided to omit a release note given these seem like edge cases.Fixes: #156511.
Fixes: #156611.
Release note: None