Skip to content

DATAMONGO-1404 DATAMONGO-1412 Add support for $min and $max update operators. #359

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

Closed
wants to merge 6 commits into from

Conversation

mp911de
Copy link
Member

@mp911de mp911de commented Apr 8, 2016

This PR is to review changes from #353 and back the changes by documenting rules for Java to MongoDB representation mapping


Related tickets: DATAMONGO-1404 DATAMONGO-1412
Related pull request: #353

odiszapc and others added 5 commits April 8, 2016 09:02
Original pull request: #353.
CLA: 169820160330091912 (Alexey Plotnik)
Add author and since tags. Update license headers. Reformat code. Replace FQCN with import and simple class name. Remove final keyword in test methods. Add tests for numeric values. Update documentation.

Original pull request: #353.
Fixed broken highlighting using backticks followed by chars/single quotes. Convert single quote emphasis of id to backtick code fences. Add missing spaces between words and backticks.
@mp911de mp911de force-pushed the issue/DATAMONGO-1404 branch from d028c07 to 17f8ae5 Compare April 26, 2016 06:59
|===
| Type
| Type conversion
| MongoDB representation
Copy link
Member

Choose a reason for hiding this comment

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

Should we rather call this Sample as the actual representation of a GeoJsonPoint would just be { "type" : "Point" , "coordinates" : [3.0 , 4.0] }.

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.

Rename MongoDB representation to sample. Represent Point within a field. Add missing whitespaces to improve readability.
christophstrobl pushed a commit that referenced this pull request May 2, 2016
…resentation.

Original Pull Request: #359
Related pull request: #353
Related ticket: DATAMONGO-1404
christophstrobl pushed a commit that referenced this pull request May 2, 2016
Fixed broken highlighting using backticks followed by chars/single quotes. Convert single quote emphasis of id to backtick code fences. Add missing spaces between words and backticks.

Original Pull Request: #359
christophstrobl pushed a commit that referenced this pull request May 2, 2016
…resentation.

Original Pull Request: #359
Related pull request: #353
Related ticket: DATAMONGO-1404
christophstrobl pushed a commit that referenced this pull request May 2, 2016
Fixed broken highlighting using backticks followed by chars/single quotes. Convert single quote emphasis of id to backtick code fences. Add missing spaces between words and backticks.

Original Pull Request: #359
christophstrobl pushed a commit that referenced this pull request May 2, 2016
…resentation.

Original Pull Request: #359
Related pull request: #353
Related ticket: DATAMONGO-1404
christophstrobl pushed a commit that referenced this pull request May 2, 2016
Fixed broken highlighting using backticks followed by chars/single quotes. Convert single quote emphasis of id to backtick code fences. Add missing spaces between words and backticks.

Original Pull Request: #359
@christophstrobl christophstrobl deleted the issue/DATAMONGO-1404 branch May 2, 2016 12:50
christophstrobl pushed a commit that referenced this pull request May 3, 2016
…resentation.

Original Pull Request: #359
Related pull request: #353
Related ticket: DATAMONGO-1404
christophstrobl pushed a commit that referenced this pull request May 3, 2016
Fixed broken highlighting using backticks followed by chars/single quotes. Convert single quote emphasis of id to backtick code fences. Add missing spaces between words and backticks.

Original Pull Request: #359
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants