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

JCR-4085 - always include property type data for DOUBLE #50

Open
wants to merge 7,690 commits into
base: trunk
Choose a base branch
from

Conversation

justinedelson
Copy link
Contributor

@justinedelson justinedelson commented Dec 14, 2016

As discussed in https://issues.apache.org/jira/browse/JCR-4085, JSON only has a single numeric type, so type inference cannot be used to differentiate between whole numbers in a LONG and DOUBLE field. To compensate for this and allow clients to properly understand the property types, DOUBLE properties should always contain type information, as is already done for DECIMAL types.

amitj and others added 30 commits October 17, 2014 07:30
JCR-3817: [jackrabbit-aws-ext] Performance of operation degrades while running DS GC

Applying patch from Shashank Gupta

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1632482 13f79535-47bb-0310-9956-ffa450edef68
Depend on Oak 1.1-SNAPSHOT for now until 1.1.2 or newer is available

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1633758 13f79535-47bb-0310-9956-ffa450edef68
LoginModule configurations without an explicit principal provider class name will now be applied to the DefaultPrincipalProvider.


git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1634584 13f79535-47bb-0310-9956-ffa450edef68
…rts, replacing Session#checkPermission by #hasPermission

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1640344 13f79535-47bb-0310-9956-ffa450edef68
…acing Session.checkPermission by Session.hasPermission

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1640530 13f79535-47bb-0310-9956-ffa450edef68
…dHashMap#remove, however LocalCache relies on that when it overrides the #remove method.

Note that this issue did not turn up during running the tests because TestLocalCache#testAutoPurge is disabled due to JCR-3806...

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1643091 13f79535-47bb-0310-9956-ffa450edef68
…runOrder configuration option. This is needed because the default order in which the tests are executed in the jackrabbit-spi2jcr module causes test failures on Java 8

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1643781 13f79535-47bb-0310-9956-ffa450edef68
Add missing license headers

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1643812 13f79535-47bb-0310-9956-ffa450edef68
Use released version of Oak

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1643994 13f79535-47bb-0310-9956-ffa450edef68
…ts metadata is accessed from S3

Patch from Shashank

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1644549 13f79535-47bb-0310-9956-ffa450edef68
…h to the number of slots available

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1644665 13f79535-47bb-0310-9956-ffa450edef68
…h to the number of slots available

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1644668 13f79535-47bb-0310-9956-ffa450edef68
…ainey jallow, commit with minor modifications and some TODOs (in particular testing).

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1646435 13f79535-47bb-0310-9956-ffa450edef68
…re Case Mode (patch provided by dominique jaeggi)

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1653275 13f79535-47bb-0310-9956-ffa450edef68
…very as discussed during f2f meeting with alfusainey and david, january 21)

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1653976 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1656043 13f79535-47bb-0310-9956-ffa450edef68
reschke and others added 26 commits December 4, 2016 12:32
…asses for better future re-use

Make internal method private.

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772530 13f79535-47bb-0310-9956-ffa450edef68
…asses for better future re-use

Make throws clauses less verbose.

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772544 13f79535-47bb-0310-9956-ffa450edef68
….testAutoPurge

Patch from Wonsan Ko
* Wait until purge job finishes
* reduce sleep to 500 ms

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772597 13f79535-47bb-0310-9956-ffa450edef68
….testAutoPurge

Increased sleep to back to 1000 ms (saw a failure in 15-20 runs)

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772822 13f79535-47bb-0310-9956-ffa450edef68
…enousRunGC

whitespace, typos, slightly improved diagnostics

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1773332 13f79535-47bb-0310-9956-ffa450edef68
… bundle plugin

add explicit version numbers for the affected packages, freezing them at 2.13.5

git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1774021 13f79535-47bb-0310-9956-ffa450edef68
@reschke
Copy link
Contributor

reschke commented Dec 14, 2016

Any chance to add a few unit tests?

@justinedelson
Copy link
Contributor Author

added unit test for this behavior. My preference would have been to add a dependency to something like https://github.com/skyscreamer/JSONassert, but it seemed like overkill to add an additional test dependency just for this one case.

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.