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

Hsearch 1442 - @IndexedEmbedded depth analysis is broken #523

Closed
wants to merge 2 commits into from

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Dec 10, 2013

My coworker Yoann Rodière worked on a test case and a fix for HSEARCH-1442. This PR is for branch 4.4 as it's the current stable branch but it should be mergeable in all supported branches.

Feedback welcome.

…s some @IndexedEmbedded with includePath set and some with depth set
…g parsed an IndexedEmbedded annotation (especially when it has been set to 0)
@Hibernate-CI
Copy link
Contributor

Merged build triggered.

1 similar comment
@Hibernate-CI
Copy link
Contributor

Merged build triggered.

@Hibernate-CI
Copy link
Contributor

Merged build started.

@Hibernate-CI
Copy link
Contributor

Merged build triggered.

@Hibernate-CI
Copy link
Contributor

Merged build finished.

@Sanne
Copy link
Member

Sanne commented Dec 10, 2013

the CI machine failed it because of some checkstyle rules but don't worry for those it's a trivial change I'll do when merging

@ghost ghost assigned hferentschik Dec 11, 2013
@hferentschik
Copy link
Contributor

To start with some administrative things, @fenrhil, have you already "signed" the JBoss CLA for Hibernate Search?

@yrodiere
Copy link
Member

Hi Hardy,

I did sign the CLA on Dec 5, 2012. It was approved on Feb 6, 2013.

@hferentschik
Copy link
Contributor

@fenrhil, thanks

@hferentschik
Copy link
Contributor

I merged this fix to 4.4, 4.5 and master. Good catch @fenrhil :-)
For me this fix mainly reverts to the previous status quo. I agree with @gsmet that this solution does not seem 100% accurate, but by rewriting it, I might introduce another bug or different behaviour.

@Sanne
Copy link
Member

Sanne commented Dec 13, 2013

thanks all!
expect to see releases from both 4.5 and 4.4 branches next week, or with some luck I'll start over the weekend.

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