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

Resolved issue #6: Stream query in release mode. #7

Merged
merged 7 commits into from
Feb 14, 2024
Merged

Resolved issue #6: Stream query in release mode. #7

merged 7 commits into from
Feb 14, 2024

Conversation

marvin-hansen
Copy link
Contributor

Describe your changes

Resolved previous issue #6

  • Added uncompressed client to handle streams in release mode
  • Added new method fetch_stream

Issue ticket number and link

#6

Code checklist before requesting a review

  • All tests are passing when running make test?
  • No errors or security vulnerabilities are reported by make check?

For details on make, please see BUILD.md

Note: The CI runs all of the above and fixing things before they hit CI speeds
up the review and merge process. Thank you.

Signed-off-by: Marvin Hansen <marvin.hansen@gmail.com>
Signed-off-by: Marvin Hansen <marvin.hansen@gmail.com>
…ming data in release mode.

Added new method fetch_stream to ,well, fetch data from a continuous query of a stream.

Signed-off-by: Marvin Hansen <marvin.hansen@gmail.com>
@marvin-hansen marvin-hansen self-assigned this Feb 14, 2024
Signed-off-by: Marvin Hansen <marvin.hansen@gmail.com>
@marvin-hansen
Copy link
Contributor Author

@jovezhong

Please review & merge. Also, please merge PR #5 afterwards to trigger the release of a new version on crates.io.

@jovezhong
Copy link
Contributor

@zliang-min will be a better person to review the Rust code (I'll take care of the docs and release)

Copy link
Collaborator

@zliang-min zliang-min left a comment

Choose a reason for hiding this comment

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

LGTM, just a couple small comments. Consider updating the query example as well?

src/lib/stream.rs Outdated Show resolved Hide resolved
src/lib/mod.rs Outdated Show resolved Hide resolved
jovezhong and others added 2 commits February 14, 2024 09:56
Co-authored-by: Gimi Liang <gimi.liang@timeplus.io>
Co-authored-by: Gimi Liang <gimi.liang@timeplus.io>
Copy link
Contributor

@jovezhong jovezhong left a comment

Choose a reason for hiding this comment

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

Thank you Gimi and Marvin for the refinement

@jovezhong jovezhong merged commit 642b60f into timeplus-io:main Feb 14, 2024
3 checks passed
@github-actions github-actions bot mentioned this pull request Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants