----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66636/ -----------------------------------------------------------
Review request for sentry, Alexander Kolbasov, Brian Towles, kalyan kumar kalvagadda, Na Li, Steve Moist, Sergio Pena, Vadim Spector, and Xinran Tinney. Repository: sentry Description ------- Since 2.0, the datanucleus version in Sentry changed from 3.2 to 4.1. Based on the datanucleus documentation following config names are renamed. Sentry has several places use those configuration names. We should update the config names to reflect the correct names. Migration from 3.3.7 to 4.0.0.M1 Migrating will require the following changes Persistence property datanucleus.allowAttachOfTransient now defaults to true for JPA usage; set it explicitly to get old behaviour Persistence property datanucleus.metadata.validate was removed (replaced by datanucleus.metadata.xml.validate some time back) Persistence property datanucleus.defaultInheritanceStrategy is renamed to datanucleus.metadata.defaultInheritanceStrategy Persistence property datanucleus.autoCreateSchema is renamed to datanucleus.schema.autoCreateAll Persistence property datanucleus.autoCreateTables is renamed to datanucleus.schema.autoCreateTables Persistence property datanucleus.autoCreateColumns is renamed to datanucleus.schema.autoCreateColumns Persistence property datanucleus.autoCreateConstraints is renamed to datanucleus.schema.autoCreateConstraints Persistence property datanucleus.validateSchema is renamed to datanucleus.schema.validateAll Persistence property datanucleus.validateTables is renamed to datanucleus.schema.validateTables Persistence property datanucleus.validateColumns is renamed to datanucleus.schema.validateColumns Persistence property datanucleus.validateConstraints is renamed to datanucleus.schema.validateConstraints Persistence property datanucleus.fixedDatastore is now removed, since it only equated to setting the "autoCreate" properties to false. Diffs ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java 83c0fc47 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java 7e02874b sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/TestSentryRole.java 61a74c3f sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java 9b0aeb2f Diff: https://reviews.apache.org/r/66636/diff/1/ Testing ------- mvn clean install all SUCCESS Thanks, Xinran Tinney
