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

ISPN-7183 Upgrade to protostream-4.0.0.Alpha2 #4655

Merged
merged 1 commit into from Nov 11, 2016

Conversation

anistor
Copy link
Member

@anistor anistor commented Nov 10, 2016

  • adapt to the protostream configuration API changes
  • update protostream marshaller for the new Transaction.longDescription property in the test domain model
  • add protostream annotations for analyzer
  • rename protostream annotation @IndexedField to @Field and enhance it with more capabilities similar to Hibernate Search

https://issues.jboss.org/browse/ISPN-7183

@anistor anistor added this to the 9.0.0.Beta1 milestone Nov 10, 2016
@gustavocoding
Copy link

There are test failures:

ProtobufMetadataManagerInterceptorTest
ProtobufWrapperIndexingTest


/**
* All fields of Protobuf types are indexed and stored by default. This behaviour is usually acceptable in most cases
* but it can become a performance problem if there are many or very large fields. To avoid such problems Infinispan allows you
Copy link
Member Author

Choose a reason for hiding this comment

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

This doc needs to be revised. I'm on it!

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

@anistor
Copy link
Member Author

anistor commented Nov 10, 2016

@gustavonalle This PR is a single commit extracted from a larger PR that is also coming soon. I might have not extracted all the changes, hence the test failures. Will fix ASAP.

* adapt to the protostream configuration API changes
* update protostream marshaller for the new Transaction.longDescription property in the test domain model
* add protostream annotations for analyzer
* rename protostream annotation @IndexedField to @field and enhance it with more capabilities similar to Hibernate Search
@anistor anistor changed the title ISPN-7183 Upgrade to protostream-4.0.0.Alpha1 ISPN-7183 Upgrade to protostream-4.0.0.Alpha2 Nov 10, 2016
@anistor
Copy link
Member Author

anistor commented Nov 10, 2016

It seems I've introduced a bug in protostream. I'll release Alpha2 to fix that today.

@anistor
Copy link
Member Author

anistor commented Nov 10, 2016

Should be fixed now.

@gustavocoding gustavocoding merged commit 172e9fe into infinispan:master Nov 11, 2016
@gustavocoding
Copy link

integrated, thanks

@anistor anistor deleted the t_7183_m branch November 11, 2016 10:10
@gustavocoding
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants