[hibernate/hibernate-validator] 0a199d: HV-1438 Use Set consistently while manipulating va...
by GitHub
Branch: refs/heads/master
Home: https://github.com/hibernate/hibernate-validator
Commit: 0a199d59cd8220d08b6a8e2f6794804d82bb37f8
https://github.com/hibernate/hibernate-validator/commit/0a199d59cd8220d08...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/valueextraction/ValueExtractorManager.java
Log Message:
-----------
HV-1438 Use Set …
[View More]consistently while manipulating value extractor
descriptors
Commit: cffcdec53fa0a27737caebbc137956c370188e2e
https://github.com/hibernate/hibernate-validator/commit/cffcdec53fa0a2773...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/xml/ConstrainedExecutableBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/xml/ConstrainedFieldBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/xml/ConstrainedGetterBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/xml/ConstrainedParameterBuilder.java
Log Message:
-----------
HV-1438 Fix XML cascading metadata for arrays
It was fixed for the annotations as part of HV-1384 but it was forgotten
for the XML part. The issue was hidden by how cascading was
implemented but some tests will fail once the follow up changes will be
incorporated.
Commit: e4d1c3138661fa6d2d43f11eebcc53e47f1873ff
https://github.com/hibernate/hibernate-validator/commit/e4d1c3138661fa6d2...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/core/MetaConstraints.java
Log Message:
-----------
HV-1438 Use the right method to throw the exception
This issue was previously hidden but it was an oversight.
Commit: 662b1605d16358dd98360e22f5df3faadd802f9f
https://github.com/hibernate/hibernate-validator/commit/662b1605d16358dd9...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/valueextraction/ValueExtractorDescriptor.java
M engine/src/main/java/org/hibernate/validator/internal/util/logging/Log.java
Log Message:
-----------
HV-1438 Improve a log message
Commit: 99c1d260697dd9857ff6be66d2d80c59c8d82d25
https://github.com/hibernate/hibernate-validator/commit/99c1d260697dd9857...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
R engine/src/main/java/org/hibernate/validator/internal/engine/valueextraction/ObjectValueExtractor.java
M engine/src/main/java/org/hibernate/validator/internal/engine/valueextraction/ValueExtractorManager.java
Log Message:
-----------
HV-1438 Remove the ObjectValueExtractor
It's one more value extractor to test and it's always a valid candidate.
Moreover, it hid some unexpected behaviors.
Commit: 68f1332356096bdb0aff919df6a7f9b62d1d8099
https://github.com/hibernate/hibernate-validator/commit/68f1332356096bdb0...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 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/valueextraction/ValueExtractorManager.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/CascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ExecutableMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/FieldCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/GetterCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ParameterMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/PropertyMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/util/Contracts.java
M engine/src/main/java/org/hibernate/validator/internal/util/logging/Log.java
Log Message:
-----------
HV-1438 Narrow down the possible value extractors to the ones compatible
with the declared type when dealing with cascaded validation
Commit: 7fbed6624d6b782b90ffaa5890d944a281f3282c
https://github.com/hibernate/hibernate-validator/commit/7fbed6624d6b782b9...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/CascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/GroupConversionHelper.java
Log Message:
-----------
HV-1437 Share the default empty instance of GroupConversionHelper
Commit: 24ae78ecf9827f032cc9892cb01d74f6902f99b3
https://github.com/hibernate/hibernate-validator/commit/24ae78ecf9827f032...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ExecutableMetaData.java
Log Message:
-----------
HV-1437 Make the signatures an immutable set
In the default case of one signature, it improves the memory usage as
the Set is converted to a singleton set.
Commit: d0e3eeb42fca2f4091b8995120c51baa2c214c62
https://github.com/hibernate/hibernate-validator/commit/d0e3eeb42fca2f409...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
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/main/java/org/hibernate/validator/internal/metadata/BeanMetaDataManager.java
M engine/src/test/java/org/hibernate/validator/test/internal/engine/path/PathImplTest.java
M engine/src/test/java/org/hibernate/validator/test/internal/metadata/BeanMetaDataManagerTest.java
M engine/src/test/java/org/hibernate/validator/test/internal/metadata/aggregated/ExecutableMetaDataTest.java
M engine/src/test/java/org/hibernate/validator/test/internal/metadata/aggregated/ParameterMetaDataTest.java
M engine/src/test/java/org/hibernate/validator/test/internal/metadata/aggregated/PropertyMetaDataTest.java
Log Message:
-----------
HV-1437 Share a common ValidationOrderGenerator
There is no point in having 2 separate ones in ValidatorImpl and
BeanMetaDataManager.
Commit: 744e679c0dfad4e0905cdc2a8b6d94cf2462ab83
https://github.com/hibernate/hibernate-validator/commit/744e679c0dfad4e09...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/CascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ExecutableMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/FieldCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/GetterCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ParameterMetaData.java
Log Message:
-----------
HV-1437 Add constant CascadingMetaDatas for non container configuration
Commit: aafb64a389c212deadb94ca0849e27eb23e664aa
https://github.com/hibernate/hibernate-validator/commit/aafb64a389c212dea...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 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-1439 Remove the iterative versions of the benchmarks: they are
useless
Commit: d0d554534b31caa6ef3d29bb94742997d0221851
https://github.com/hibernate/hibernate-validator/commit/d0d554534b31caa6e...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M performance/README.md
M performance/pom.xml
Log Message:
-----------
HV-1439 Adjust how the performance tests are run
Commit: de37a14df62eb939be30b32e839f30c519a661fc
https://github.com/hibernate/hibernate-validator/commit/de37a14df62eb939b...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M performance/README.md
M performance/pom.xml
Log Message:
-----------
HV-1439 Update 5.4 to 5.4.1 in performance tests
Commit: 9d893078436291f1789f0e94d7a6431750c4043e
https://github.com/hibernate/hibernate-validator/commit/9d893078436291f17...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
Log Message:
-----------
HV-1439 Reduce the contention on the beanMetaDataCache concurrent map
I introduced the infrastructure for that quite some time ago but only
used it for properties, whereas it's also beneficial in the method
validation path.
Commit: 485750cce9f797bc0776c109539e930d8113cf34
https://github.com/hibernate/hibernate-validator/commit/485750cce9f797bc0...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/util/Contracts.java
M engine/src/main/java/org/hibernate/validator/internal/util/TypeHelper.java
Log Message:
-----------
HV-1439 Avoid using toString() and string append if we don't have to log anything
Commit: efdd5115803725f2c526a67ecae574a8ddb1f67a
https://github.com/hibernate/hibernate-validator/commit/efdd5115803725f2c...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ReturnValueMetaData.java
Log Message:
-----------
HV-1439 Optimize a bit the initialization of ReturnValueMetaData
Commit: a98ce3d84800f99c9766c410de30d4432f427e74
https://github.com/hibernate/hibernate-validator/commit/a98ce3d84800f99c9...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/provider/AnnotationMetaDataProvider.java
Log Message:
-----------
HV-1439 Use computeIfAbsent when retrieving BeanConfiguration
Commit: 1749f69a5589f203e0aa96644b2055bbbb442ce2
https://github.com/hibernate/hibernate-validator/commit/1749f69a5589f203e...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/CascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ExecutableMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/FieldCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/GetterCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/GroupConversionHelper.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ParameterMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ReturnValueMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/cascading/CascadingTypeParameter.java
M engine/src/main/java/org/hibernate/validator/internal/util/logging/Log.java
Log Message:
-----------
HV-1441 Clean up the initialization of CascadingMetaData
Commit: 08f46dc32506ec51c0b5257d9daca21a6c896bae
https://github.com/hibernate/hibernate-validator/commit/08f46dc32506ec51c...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/AbstractConstraintMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/CascadingMetaData.java
A engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ContainerCascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ExecutableMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/FieldCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/GetterCascadable.java
A engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/NonContainerCascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ParameterMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/cascading/CascadingTypeParameter.java
M engine/src/main/java/org/hibernate/validator/internal/util/logging/Log.java
Log Message:
-----------
HV-1441 Separate CascadingMetaData in 2 classes: one very lightweight
for non containers and the other full featured for containers
Commit: e0f480f53110e7ca062a975cf3eb0decaf365f82
https://github.com/hibernate/hibernate-validator/commit/e0f480f53110e7ca0...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/cfg/context/CascadableConstraintMappingContextImplBase.java
M engine/src/main/java/org/hibernate/validator/internal/cfg/context/ContainerElementConstraintMappingContextImpl.java
M engine/src/main/java/org/hibernate/validator/internal/cfg/context/ExecutableConstraintMappingContextImpl.java
M engine/src/main/java/org/hibernate/validator/internal/cfg/context/ParameterConstraintMappingContextImpl.java
M engine/src/main/java/org/hibernate/validator/internal/cfg/context/PropertyConstraintMappingContextImpl.java
A engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/CascadingMetaDataBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ContainerCascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ExecutableMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/FieldCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/GetterCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/MetaDataBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/NonContainerCascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ParameterMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/PropertyMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/rule/ParallelMethodsMustNotDefineGroupConversionForCascadedReturnValue.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/rule/ReturnValueMayOnlyBeMarkedOnceAsCascadedPerHierarchyLine.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/rule/VoidMethodsMustNotBeReturnValueConstrained.java
R engine/src/main/java/org/hibernate/validator/internal/metadata/cascading/CascadingTypeParameter.java
R engine/src/main/java/org/hibernate/validator/internal/metadata/cascading/package-info.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/facets/Cascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/provider/AnnotationMetaDataProvider.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/AbstractConstrainedElement.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/ConstrainedElement.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/ConstrainedExecutable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/ConstrainedField.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/ConstrainedParameter.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/ConstrainedType.java
M engine/src/main/java/org/hibernate/validator/internal/xml/ConstrainedExecutableBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/xml/ConstrainedFieldBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/xml/ConstrainedGetterBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/xml/ConstrainedParameterBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/xml/ContainerElementTypeConfigurationBuilder.java
M engine/src/test/java/org/hibernate/validator/test/internal/metadata/provider/AnnotationMetaDataProviderTest.java
Log Message:
-----------
HV-1441 Rename CascadingTypeParameter to CascadingMetaDataBuilder
It does not exactly follow the builder pattern but it's really what it
does.
Commit: 99551f9c5c6a1e20ab07ddbd53726a5f75ae4ff9
https://github.com/hibernate/hibernate-validator/commit/99551f9c5c6a1e20a...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/AbstractConstraintMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/CascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/CascadingMetaDataBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ContainerCascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ExecutableMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/MetaDataBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/NonContainerCascadingMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ParameterMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/PropertyMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ReturnValueMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/rule/ParallelMethodsMustNotDefineGroupConversionForCascadedReturnValue.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/rule/ReturnValueMayOnlyBeMarkedOnceAsCascadedPerHierarchyLine.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/rule/VoidMethodsMustNotBeReturnValueConstrained.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/AbstractConstrainedElement.java
Log Message:
-----------
HV-1441 Some naming improvements to improve consistency
Commit: 39e70a9499c89dd24fa453212eced860afe50d50
https://github.com/hibernate/hibernate-validator/commit/39e70a9499c89dd24...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/CascadingMetaDataBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ContainerCascadingMetaData.java
Log Message:
-----------
HV-1439 Have a specific method to check that cascading should be
performed on container elements: there is no need to execute part of the
code if only the annotated object is cascaded.
Commit: ca0a8041bfad5f7c1bfc4eeb27c96c0fc19bbc8c
https://github.com/hibernate/hibernate-validator/commit/ca0a8041bfad5f7c1...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorFactoryImpl.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/core/ConstraintHelper.java
Log Message:
-----------
HV-1439 Detect built-in constraints right away and keep a cache of the detected constraints
Commit: 24f21d1a522854d5024c5b7a734a01063f88fbd6
https://github.com/hibernate/hibernate-validator/commit/24f21d1a522854d50...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/MethodValidationConfiguration.java
Log Message:
-----------
HV-1439 Create only once the set of rules as everything is immutable
Commit: 00ba073e9d69a9dc0b4a795c0574bcda46c3fa46
https://github.com/hibernate/hibernate-validator/commit/00ba073e9d69a9dc0...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/BeanMetaDataImpl.java
Log Message:
-----------
HV-1439 Add hashCode() and equals() methods to the builder delegate as it's
used in a HashSet
Commit: 567cbe92502111a16bdef54a6d64a68319b23f46
https://github.com/hibernate/hibernate-validator/commit/567cbe92502111a16...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/ExecutableMetaData.java
Log Message:
-----------
HV-1437 No need to copy the set of rules as it's immutable
Commit: 82167c74d570f188c5abf28f729c1bb8839eef62
https://github.com/hibernate/hibernate-validator/commit/82167c74d570f188c...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/provider/AnnotationMetaDataProvider.java
Log Message:
-----------
HV-1439 Avoid calling reflection methods multiple times
Commit: 06d6fd2982b61f3061f15d1e9d0f71fb51799a3c
https://github.com/hibernate/hibernate-validator/commit/06d6fd2982b61f306...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/descriptor/ConstraintDescriptorImpl.java
A engine/src/main/java/org/hibernate/validator/internal/util/privilegedactions/GetAnnotationParameters.java
Log Message:
-----------
HV-1439 Get all the annotation parameters at once
Getting them one by one is excessively costly.
Commit: 25d2e6e3e097494f3ba33fb04a4be0dfb26e6549
https://github.com/hibernate/hibernate-validator/commit/25d2e6e3e097494f3...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/descriptor/ConstraintDescriptorImpl.java
Log Message:
-----------
HV-1439 Avoid calling toImmutable in the getter: the collection is already immutable
Commit: f31db806faa59d2253daab25edc593c142519f0d
https://github.com/hibernate/hibernate-validator/commit/f31db806faa59d225...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/util/annotationfactory/AnnotationProxy.java
Log Message:
-----------
HV-1439 Make the AnnotationProxy equals more efficient
The current method is calling GetDeclaredMethod for each attribute of
the annotation, which is not a good idea, considering this call is
rather slow.
Instead, if the annotation is an AnnotationProxy, we directly work on
the map that is in the InvocationHandler, and if not, we get all the
parameters at once.
Commit: d86d336800bd94b7291e3832d3eb5231e68dc743
https://github.com/hibernate/hibernate-validator/commit/d86d336800bd94b72...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/PropertyMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/AbstractConstrainedElement.java
Log Message:
-----------
HV-1443 AbstractConstrainedElement#isConstrained() should also return true if there are group conversions without cascading
Commit: e676956ff796af718ee98016640af41a21d789b4
https://github.com/hibernate/hibernate-validator/commit/e676956ff796af718...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/provider/AnnotationMetaDataProvider.java
Log Message:
-----------
HV-1439 Avoid calling getParameters() if there are no parameters
Commit: 6ea2fc22bfc115d4ea3612b86aaa6f170b842dd6
https://github.com/hibernate/hibernate-validator/commit/6ea2fc22bfc115d4e...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/GetterCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/MetaDataBuilder.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/PropertyMetaData.java
Log Message:
-----------
HV-1439 HV-1437 Only create the accessible method once and share the
GetterConstraintLocation
Commit: b52f1fbc78a0fce3f38562717cab7eb1ad1a7286
https://github.com/hibernate/hibernate-validator/commit/b52f1fbc78a0fce3f...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/FieldCascadable.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/PropertyMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/location/FieldConstraintLocation.java
M engine/src/main/java/org/hibernate/validator/internal/util/privilegedactions/GetDeclaredField.java
M engine/src/main/java/org/hibernate/validator/internal/util/privilegedactions/GetDeclaredMethod.java
M engine/src/main/java/org/hibernate/validator/internal/util/privilegedactions/SetAccessibility.java
Log Message:
-----------
HV-1439 Group the reflection and the setAccessible priviledged actions
Commit: 6be092872ef8f23165db340cd24541577a5d8275
https://github.com/hibernate/hibernate-validator/commit/6be092872ef8f2316...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/metadata/core/ConstraintHelper.java
Log Message:
-----------
HV-1439 Add a comment on why we don't use isJdkAnnotation() in isConstraintAnnotation()
Commit: 6caa16d0b1402c75e44bb9cbad3dbf70578e474c
https://github.com/hibernate/hibernate-validator/commit/6caa16d0b1402c75e...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/util/annotationfactory/AnnotationProxy.java
Log Message:
-----------
HV-1439 Disable AnnotationProxy equals optimization if the security manager is enabled
Apparently, the security check for getInvocationHandler() was not
correctly implemented in JDK 8 < 131.
Commit: aa03d82464157fac5ee535e6032d2768204b71f1
https://github.com/hibernate/hibernate-validator/commit/aa03d82464157fac5...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/BeanMetaData.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/BeanMetaDataImpl.java
M engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/UnconstrainedEntityMetaDataSingleton.java
M engine/src/main/java/org/hibernate/validator/internal/util/logging/Log.java
M engine/src/test/java/org/hibernate/validator/test/internal/engine/methodvalidation/WrongMethodOrParameterForMethodValidationTest.java
M engine/src/test/java/org/hibernate/validator/test/internal/engine/path/PathImplTest.java
M engine/src/test/java/org/hibernate/validator/test/internal/metadata/CustomerRepositoryExt.java
M engine/src/test/java/org/hibernate/validator/test/internal/metadata/aggregated/ExecutableMetaDataTest.java
M engine/src/test/java/org/hibernate/validator/test/internal/metadata/aggregated/JobRepository.java
M engine/src/test/java/org/hibernate/validator/test/internal/metadata/aggregated/ParameterMetaDataTest.java
M engine/src/test/java/org/hibernate/validator/test/internal/metadata/aggregated/SpecialJobRepositoryImpl.java
Log Message:
-----------
HV-1447 Remove metadata for unconstrained methods from the aggregated metamodel
We still keep a set of the signatures of the unconstained methods to be
able to throw an exception if the method does not exist.
Commit: 51ca465d2ac88aaf76abc1243a957c71034eb3f1
https://github.com/hibernate/hibernate-validator/commit/51ca465d2ac88aaf7...
Author: Guillaume Smet <guillaume.smet(a)gmail.com>
Date: 2017-08-03 (Thu, 03 Aug 2017)
Changed paths:
M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
Log Message:
-----------
HV-1447 Be more consistent about the naming of validationContext
context is not clear enough as we also have the valueContext.
Compare: https://github.com/hibernate/hibernate-validator/compare/9d6c5b80aac3...5...
[View Less]
7 years, 5 months