-
Notifications
You must be signed in to change notification settings - Fork 211
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
justinedelson
wants to merge
7,690
commits into
apache:trunk
Choose a base branch
from
justinedelson:issue/JCR-4085
base: trunk
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1633070 13f79535-47bb-0310-9956-ffa450edef68
This closes apache#20 git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1633086 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
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1636607 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
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1640531 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1640892 13f79535-47bb-0310-9956-ffa450edef68
…oduced by angela git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1640897 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
…va 8 git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1643520 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1643522 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
…gistering node types git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1644405 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
…time Patch from Shashank git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1644551 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
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1647241 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
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1653309 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1653930 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1653935 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
…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
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772545 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772548 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772549 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772551 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772553 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772558 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772562 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772563 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772565 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772566 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772569 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1772571 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
(ack Woonsan Ko) git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1773303 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
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1773579 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1773591 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1773745 13f79535-47bb-0310-9956-ffa450edef68
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1773809 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
git-svn-id: https://svn.apache.org/repos/asf/jackrabbit/trunk@1774034 13f79535-47bb-0310-9956-ffa450edef68
Any chance to add a few unit tests? |
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.