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.

Sign up for GitHub

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

Jump to bottom

Infinispan Client should use ProtoStreamMarshaller as default #798

Closed
wburns opened this issue Feb 7, 2019 · 2 comments
Closed

Infinispan Client should use ProtoStreamMarshaller as default #798

wburns opened this issue Feb 7, 2019 · 2 comments
Assignees
@wburns
Labels
kind/enhancement New feature or request
Milestone
0.9.1

Comments

@wburns
Copy link
Member

wburns commented Feb 7, 2019

As has been discussed in #786 we should change it to have a better default marshaller. The obvious choice is proto stream marshaller as this reduces additional configuration on the part of the user.

We would need to add the remote query module as a dependency and update code and integration tests to this effect.

The docs also need to be gutted to not talk about configuring the marshaller any longer and just talk about user type serialization.
@wburns wburns mentioned this issue Feb 7, 2019
Merged
@Sanne Sanne added the kind/enhancement New feature or request label Feb 7, 2019
@wburns wburns mentioned this issue Feb 7, 2019
Merged
@wburns
Copy link
Member Author

wburns commented Feb 14, 2019

Integrated with #807

@wburns wburns closed this as completed Feb 14, 2019
@Sanne Sanne added this to the 0.9.1 milestone Feb 14, 2019
@Sanne
Copy link
Member

Sanne commented Feb 14, 2019

@cescoffier we failed to notice that this should have been flagged for milestone 0.9.0 .. what's the process now, ignore? I merked it for 0.9.1 now, hopefully that's ok..

maxandersen added a commit to maxandersen/quarkus that referenced this issue Nov 5, 2022
@maxandersen
fix: be able to launch editors with space in name (quarkusio#800)
b2b0662 
Affects all Operating systems.
Now forcefully escape launched editor.

Fix quarkusio#798
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wburns wburns

Labels
kind/enhancement New feature or request
Projects
None yet
Milestone
0.9.1
Development

No branches or pull requests
2 participants
@Sanne @wburns