Skip to content
This repository was archived by the owner on Feb 8, 2019. It is now read-only.

BLUR-341 fixed terms not working for numeric fields#2

Open
gbarton wants to merge 5 commits intoapache:masterfrom
gbarton:master
Open

BLUR-341 fixed terms not working for numeric fields#2
gbarton wants to merge 5 commits intoapache:masterfrom
gbarton:master

Conversation

@gbarton
Copy link
Contributor

@gbarton gbarton commented Aug 10, 2014

Copy link
Contributor

Choose a reason for hiding this comment

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

nit-picky, but can you make the whitespace consistent with existing code (hopefully just a Ctrl+Shift+F away) for each file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed. At least I tried to, I could not determine the exact formatting style being used as everything I tried to do to the whole of IndexManager changed the whole file. Perhaps a committed style for eclipse in the blur project would help?

@Humbedooh
Copy link
Member

Testing GitHub integration once again, please do ignore :)

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 in the basefieldmanager, maybe it should be moved up? (mostly testing the github comment integration here :) )

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Going to skip this as this recommendation as it potentially gets dealt with in the blur-platform branch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants