[hibernate/hibernate-validator] 338e4c: HV-1480 Add a hv-6.0 entry to be able to compare t...
by GitHub
Branch: refs/heads/master
Home: https://github.com/hibernate/hibernate-validator
Commit: 338e4c47b6a9611e9e6dc254a000873e4df41b0c
https://github.com/hibernate/hibernate-validator/commit/338e4c47b6a9611e9...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M performance/pom.xml
Log Message:
-----------
HV-1480 Add a hv-6.0 entry to be able to compare the latest stable with
our current snapshot
Commit: 103cb5edb2d7c829f47f43871fcdc69f60f9637c
https://github.com/hibernate/hibernate-validator/commit/103cb5edb2d7c829f...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M performance/README.md
A performance/src/main/java/org/hibernate/validator/performance/cascaded/CascadedWithLotsOfItemsValidation.java
Log Message:
-----------
HV-1480 Add a benchmark validating a bean containing a lot of beans to
cascade to
Commit: 68a15fb085a9c4adde3ecddcd65a8075cc08adf9
https://github.com/hibernate/hibernate-validator/commit/68a15fb085a9c4add...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/path/PathImpl.java
Log Message:
-----------
HV-1480 Avoid removing and adding element to the node list
We can directly set the element in the list. It avoids some list
resizing.
Commit: 237aeb8945e8cbf9598782c0a8c55d56d92f171e
https://github.com/hibernate/hibernate-validator/commit/237aeb8945e8cbf95...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/path/PathImpl.java
Log Message:
-----------
HV-1480 Copy the hashCode in the copy constructor
Commit: 87affe03d84f5998072585ff0c14030c1911344a
https://github.com/hibernate/hibernate-validator/commit/87affe03d84f59980...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/groups/DefaultValidationOrder.java
M engine/src/main/java/org/hibernate/validator/internal/engine/groups/ValidationOrder.java
M engine/src/main/java/org/hibernate/validator/internal/engine/groups/ValidationOrderGenerator.java
Log Message:
-----------
HV-1480 Avoid initializing lists and maps in the common case where we
only have the default group
Also optimize a bit the advanced case with groups.
Commit: 4a053a549668d4e00dd1ce7021ab0d08ea31c6b2
https://github.com/hibernate/hibernate-validator/commit/4a053a549668d4e00...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/constraintvalidation/ConstraintValidatorContextImpl.java
Log Message:
-----------
HV-1480 Avoid creating a list in the common case when we only have the
default violation
Also optimize a bit the concatenation of the default violation and the
custom ones to avoid creating a list too small and resize it each time.
Commit: 9388c053a770dccd696ddcd314990c649efb2fb8
https://github.com/hibernate/hibernate-validator/commit/9388c053a770dccd6...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/path/NodeImpl.java
Log Message:
-----------
HV-1480 Avoid computing the hashCode if not necessary
A lot of NodeImpls are created during the build of the path and some of
them are simply discarded as they are "modified" (NodeImpl are immutable
so we create a new) further away. Thus we better avoid building the
hashCode each time.
Commit: 7546a0c14bd70d6c4b707cb9bc4d333a87891148
https://github.com/hibernate/hibernate-validator/commit/7546a0c14bd70d6c4...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/path/NodeImpl.java
Log Message:
-----------
HV-1480 Don't take into account the parent in hashCode and equals
Otherwise, we "compare" the whole path several times while comparing a
path.
Commit: 3c3354ab8f4917dc4c8ed6fc391ef741624896c8
https://github.com/hibernate/hibernate-validator/commit/3c3354ab8f4917dc4...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/path/PathImpl.java
Log Message:
-----------
HV-1480 Implement a copy on write strategy for the node list
It avoids copying the list when not strictly necessary
Commit: 710f206237078198eca3e89413449b7d6586602b
https://github.com/hibernate/hibernate-validator/commit/710f206237078198e...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidationContext.java
Log Message:
-----------
HV-1480 Centralize the processed works in one single set
It avoids a lot of initialization/resizing and in the end it's more
efficient.
Commit: 17971aa16d8e402ca6b2d2a7c421718317af3c6f
https://github.com/hibernate/hibernate-validator/commit/17971aa16d8e402ca...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
M engine/src/main/java/org/hibernate/validator/internal/engine/ValueContext.java
M engine/src/main/java/org/hibernate/validator/internal/engine/constraintvalidation/ConstraintValidatorContextImpl.java
M engine/src/main/java/org/hibernate/validator/internal/engine/path/NodeImpl.java
M engine/src/main/java/org/hibernate/validator/internal/engine/path/PathImpl.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/core/MetaConstraint.java
Log Message:
-----------
HV-1480 Setting the key or the index already makes the node iterable
So we don't need to do it twice. Note that this change uncovers the fact
that in ConstraintValidatorContext, calling atKey() or atIndex() makes
the node iterable.
It was already the case before and I think it's acceptable.
It brings its own performance improvements as it avoids initializing 1
NodeImpl and creating 1 copy of the Path.
Commit: 4b8189a3776764a7064ded00d87605e64b1ff1ce
https://github.com/hibernate/hibernate-validator/commit/4b8189a3776764a70...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/path/PathImpl.java
Log Message:
-----------
HV-1480 We expect at least one node in the path so let's initialize the
list with one element
It doesn't seem necessary to consider more elements as the list will be
copied when new nodes will be added.
Commit: 985ad8e681fcfc2634446c251562d2b6ea25410d
https://github.com/hibernate/hibernate-validator/commit/985ad8e681fcfc263...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/path/PathImpl.java
Log Message:
-----------
HV-1480 Even if not strictly necessary, the leaf node should be
correctly set in all constructors
Commit: 22dad48a2fde9d418c88c73bb867a1c41efc36a4
https://github.com/hibernate/hibernate-validator/commit/22dad48a2fde9d418...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M performance/src/main/java/org/hibernate/validator/performance/cascaded/CascadedValidation.java
M performance/src/main/java/org/hibernate/validator/performance/simple/SimpleValidation.java
Log Message:
-----------
HV-1480 Reduce the number of iterations in the other benchmarks
Commit: b74d32005a759a9df653cf2833eecc240959862c
https://github.com/hibernate/hibernate-validator/commit/b74d32005a759a9df...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M performance/src/main/java/org/hibernate/validator/performance/BenchmarkRunner.java
Log Message:
-----------
HV-1480 Add the new benchmark to the default benchmarks
Commit: bbee5ea510aec133754480663be64f70d940110f
https://github.com/hibernate/hibernate-validator/commit/bbee5ea510aec1337...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidationContext.java
Log Message:
-----------
HV-1480 Some more optimizations suggested by Sanne
Commit: f0787b516e84c914af924c7caf7a8b82201ae37e
https://github.com/hibernate/hibernate-validator/commit/f0787b516e84c914a...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M performance/README.md
A performance/src/main/java/org/hibernate/validator/performance/cascaded/CascadedWithLotsOfItemsAndMoreConstraintsValidation.java
Log Message:
-----------
HV-1480 Add another benchmark
Commit: 21975d8e5aed94cb58339f717afd92d265323a2c
https://github.com/hibernate/hibernate-validator/commit/21975d8e5aed94cb5...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/test/java/org/hibernate/validator/test/internal/engine/groups/validation/GroupValidationTest.java
Log Message:
-----------
Mark a test as candidate for the TCK
Commit: 311258b14591b16fe560ce4227d42254da7b6e07
https://github.com/hibernate/hibernate-validator/commit/311258b14591b16fe...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValueContext.java
M engine/src/main/java/org/hibernate/validator/internal/engine/path/PathImpl.java
Log Message:
-----------
HV-1480 Only set the property value if required
Only property and container node exposes it to users.
Commit: 17ec093271bdb377043418436104e7fb1aba5b69
https://github.com/hibernate/hibernate-validator/commit/17ec093271bdb3770...
Author: Gunnar Morling <gunnar.morling(a)googlemail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/path/PathImpl.java
Log Message:
-----------
HV-1480 Getting reference to parent node in a simpler way
Compare: https://github.com/hibernate/hibernate-validator/compare/93e563b240b9...1...
7 years, 1 month
[hibernate/hibernate-validator] 8b2f0d: HV-1487 If we are using a traverse all Traversable...
by GitHub
Branch: refs/heads/master
Home: https://github.com/hibernate/hibernate-validator
Commit: 8b2f0d624f90a9b0d315bca74b756595b0434ad4
https://github.com/hibernate/hibernate-validator/commit/8b2f0d624f90a9b0d...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ConfigurationImpl.java
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
R engine/src/main/java/org/hibernate/validator/internal/engine/resolver/DefaultTraversableResolver.java
M engine/src/main/java/org/hibernate/validator/internal/engine/resolver/JPATraversableResolver.java
A engine/src/main/java/org/hibernate/validator/internal/engine/resolver/TraversableResolvers.java
A engine/src/main/java/org/hibernate/validator/internal/engine/resolver/TraverseAllTraversableResolver.java
M engine/src/test/java/org/hibernate/validator/test/internal/engine/traversableresolver/JpaTraversableResolverTest.java
Log Message:
-----------
HV-1487 If we are using a traverse all TraversableResolver, don't wrap
it into a caching one
It avoids quite a lot of useless map lookups.
Originally suggested by Chris Narburgh.
Commit: b3019090e0df82c3400e0c56c3badb7ca1f3ea8e
https://github.com/hibernate/hibernate-validator/commit/b3019090e0df82c34...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/HibernateValidatorConfiguration.java
M engine/src/main/java/org/hibernate/validator/HibernateValidatorContext.java
M engine/src/main/java/org/hibernate/validator/internal/engine/ConfigurationImpl.java
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorContextImpl.java
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorFactoryImpl.java
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
M engine/src/test/java/org/hibernate/validator/test/internal/engine/traversableresolver/CachedTraversableResolverTest.java
A engine/src/test/resources/org/hibernate/validator/test/internal/engine/traversableresolver/validation-CachedTraversableResolverTest.xml
Log Message:
-----------
HV-1487 Allow to disable the TraversableResolver result cache
Commit: 49287679e85723f91e3184c3afa76fcd9ac7161f
https://github.com/hibernate/hibernate-validator/commit/49287679e85723f91...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/HibernateValidatorConfiguration.java
M engine/src/main/java/org/hibernate/validator/HibernateValidatorContext.java
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
A engine/src/main/java/org/hibernate/validator/internal/engine/resolver/CachingJPATraversableResolverForSingleValidation.java
M engine/src/main/java/org/hibernate/validator/internal/engine/resolver/CachingTraversableResolverForSingleValidation.java
M engine/src/main/java/org/hibernate/validator/internal/engine/resolver/JPATraversableResolver.java
M engine/src/main/java/org/hibernate/validator/internal/engine/resolver/TraversableResolvers.java
M engine/src/main/java/org/hibernate/validator/internal/engine/resolver/TraverseAllTraversableResolver.java
Log Message:
-----------
HV-1487 Introduce a caching wrapper specific to the
JPATraversableResolver
It allows a few optimizations based on the contract defined in the Bean
Validation specification.
Commit: 1ceb8a0b13814a162d4f505d802d8a12e507b975
https://github.com/hibernate/hibernate-validator/commit/1ceb8a0b13814a162...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
M documentation/src/main/asciidoc/ch09.asciidoc
M documentation/src/test/java/org/hibernate/validator/referenceguide/chapter09/BootstrappingTest.java
A documentation/src/test/java/org/hibernate/validator/referenceguide/chapter09/MyFastTraversableResolver.java
A documentation/src/test/java/org/hibernate/validator/referenceguide/chapter09/validation-DisableTraversableResolverResultCache.xml
Log Message:
-----------
HV-1487 Add some documentation about how to disable the
TraversableResolver cache
Commit: 93e563b240b9ce9cfec9c1e3a4c044b00f65f490
https://github.com/hibernate/hibernate-validator/commit/93e563b240b9ce9cf...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-10-18 (Wed, 18 Oct 2017)
Changed paths:
A engine/src/main/java/org/hibernate/validator/internal/engine/resolver/AbstractTraversableHolder.java
M engine/src/main/java/org/hibernate/validator/internal/engine/resolver/CachingJPATraversableResolverForSingleValidation.java
M engine/src/main/java/org/hibernate/validator/internal/engine/resolver/CachingTraversableResolverForSingleValidation.java
Log Message:
-----------
HV-1487 Avoid some code duplication
Compare: https://github.com/hibernate/hibernate-validator/compare/fc454b6b64be...9...
7 years, 1 month