Skip to content

remove JSUP#6838

Merged
mccanne merged 4 commits intomainfrom
remove-jsup
Apr 21, 2026
Merged

remove JSUP#6838
mccanne merged 4 commits intomainfrom
remove-jsup

Conversation

@mccanne
Copy link
Copy Markdown
Collaborator

@mccanne mccanne commented Apr 21, 2026

No description provided.

@philrz philrz requested a review from a team April 21, 2026 00:55
Copy link
Copy Markdown
Member

@nwt nwt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't forget these two.

$ git grep JSUP book
book/src/database/api.md:| JSUP             | yes       | yes      | `application/x-zjson`                 |
book/src/dev/libraries/python.md:# Zeek TSV, or JSUP.```

Comment thread service/ztests/curl-query.yaml Outdated
Comment on lines +1 to +2
skip: TBD replace deprecated JSUP control

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove x-jsup instead of skipping this.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Specifically:

diff --git a/service/ztests/curl-query.yaml b/service/ztests/curl-query.yaml
index bc0e70b4f..5af7428ac 100644
--- a/service/ztests/curl-query.yaml
+++ b/service/ztests/curl-query.yaml
@@ -4,10 +4,9 @@ script: |
   source service.sh
   super db create -q test
   super db load -q -use test -
-  for accept in text/csv application/{json,x-ndjson,x-sup,x-zeek,x-jsup} ""; do
+  for accept in text/csv application/{json,x-ndjson,x-sup,x-zeek} ""; do
     echo === $accept ===
-    curl -H "Accept: $accept" -d '{"query":"from test"}' $SUPER_DB/query\?ctrl=true |
-      sed -E '/QueryStats/s/[0-9]{3,}/xxx/g' # for JSUP
+    curl -H "Accept: $accept" -d '{"query":"from test"}' $SUPER_DB/query\?ctrl=true
   done
   echo === application/vnd.apache.arrow.stream ===
   curl -H 'Accept: application/vnd.apache.arrow.stream' -d '{"query":"from test"}' $SUPER_DB/query |```

@mccanne mccanne merged commit 4ad8467 into main Apr 21, 2026
4 checks passed
@mccanne mccanne deleted the remove-jsup branch April 21, 2026 01:27
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