Skip to content

Fix compression chaining with vector interceptor - #151

Closed
dkropachev wants to merge 2 commits into
scylladb:vector-1from
dkropachev:split/vector-compression-chaining
Closed

Fix compression chaining with vector interceptor#151
dkropachev wants to merge 2 commits into
scylladb:vector-1from
dkropachev:split/vector-compression-chaining

Conversation

@dkropachev

Copy link
Copy Markdown
Collaborator

Summary

  • Make request gzip compression cache compressed bytes before the AWS SDK asks for the content body, so headers and body stay consistent when vector request rewriting runs first.
  • Add regression coverage for vector-before-gzip request ordering and response decompression before vector response parsing.
  • Keep this stacked on vector-1 because the tests depend on vector-search support added there.

Base

Tests

  • JAVA_HOME=/usr/lib/jvm/java-11-openjdk-amd64 PATH=/usr/lib/jvm/java-11-openjdk-amd64/bin:$PATH mvn -q fmt:check
  • JAVA_HOME=/usr/lib/jvm/java-11-openjdk-amd64 PATH=/usr/lib/jvm/java-11-openjdk-amd64/bin:$PATH mvn -q -Dtest=VectorSearchInterceptorTest,VectorSearchHttpInterceptorTest,GzipRequestInterceptorTest,ResponseCompressionInterceptorTest test

@dkropachev

Copy link
Copy Markdown
Collaborator Author

Folded into #145 as part of the squashed review fixes.

@dkropachev dkropachev closed this Jul 29, 2026
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.

1 participant