Skip to content

Simple Spring Queries #92

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

Merged
merged 3 commits into from
Jun 20, 2020
Merged

Simple Spring Queries #92

merged 3 commits into from
Jun 20, 2020

Conversation

yschimke
Copy link
Member

@yschimke yschimke commented Jun 19, 2020

example command line

rsocket-cli --stream --route 'searchTweets' -i "{}" ws://localhost:8080/rsocket

@yschimke yschimke requested a review from linux-china June 19, 2020 20:57
@@ -152,9 +152,14 @@ class Main : Runnable {
inputPublisher = LineInputPublishers(outputHandler)
}

if (route != null) {
metadataFormat = "composite"
Copy link
Member Author

Choose a reason for hiding this comment

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

@linux-china What are firm assumptions we can make when route is selected?

Copy link
Contributor

Choose a reason for hiding this comment

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

it's a nice to make Spring route simple. A question, I know some developers use cbor for data en/decoding with RSocket. Could you comment dataFormat assign(json by default now) when route is not null, and we can use route and cbor both. thanks.

    if (route != null) {
      metadataFormat = "composite"
      // dataFormat = "json
 }

Copy link
Member Author

Choose a reason for hiding this comment

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

Should I remove binary as the default? Or check for it and then override to json?

I'd like to get it down to the minimal set of command line arguments to activate this.

Copy link
Contributor

Choose a reason for hiding this comment

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

for binary, could we consider base64 text for cbor and other binary?

Copy link
Contributor

Choose a reason for hiding this comment

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

binary as base64 text just for reference to implement CLI. we can consider this feature later. Now json is good for CLI to Spring RSocket, and Spring RSocketStrategies supports to config multi encoders.

Copy link
Member Author

Choose a reason for hiding this comment

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

OK, I'm only setting when unset. Take a look.

@yschimke yschimke merged commit 4f301fd into master Jun 20, 2020
@yschimke yschimke deleted the simple_queries branch September 12, 2020 19:01
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