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-2070 Using TypeMetadata API instead of IndexedTypeDescriptor API #958

Closed
wants to merge 1 commit into from

Conversation

@yrodiere
Copy link
Member

yrodiere commented Dec 10, 2015

No description provided.

@yrodiere yrodiere changed the title Test case for HHH-2070 (problem whith embedded date fields metadata) Test case for HSEARCH-2070 (problem whith embedded date fields metadata) Dec 10, 2015
The IndexedTypeDescriptor API does not contain information on embedded
fields which are necessary here.
@gsmet gsmet force-pushed the openwide-java:HSEARCH-2070 branch from a1373df to e101f55 Dec 16, 2015
@gsmet gsmet changed the title Test case for HSEARCH-2070 (problem whith embedded date fields metadata) @fenrhil HSEARCH-2070 Using TypeMetadata API instead of IndexedTypeDescriptor API Dec 16, 2015
@gsmet gsmet changed the title @fenrhil HSEARCH-2070 Using TypeMetadata API instead of IndexedTypeDescriptor API HSEARCH-2070 Using TypeMetadata API instead of IndexedTypeDescriptor API Dec 16, 2015
@gsmet

This comment has been minimized.

Copy link
Member

gsmet commented Dec 16, 2015

Pull request updated with a fix following @gunnarmorling 's advice.

@Sanne Sanne self-assigned this Dec 16, 2015
@Sanne

This comment has been minimized.

Copy link
Member

Sanne commented Dec 16, 2015

thanks @gsmet and @fenrhil I'm mostly away today and tomorrow but will make sure these faceting fixes are included in the next 5.5.2

@gsmet

This comment has been minimized.

Copy link
Member

gsmet commented Dec 16, 2015

Hi @Sanne!

Nice, thanks. I pushed some additional fixes for @DavideD to HSEARCH-2069.

@Sanne

This comment has been minimized.

Copy link
Member

Sanne commented Dec 21, 2015

looks great, I'm merging it. Thanks a lot!

@Sanne Sanne closed this Dec 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.