[ https://issues.apache.org/jira/browse/GEODE-8700?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17230336#comment-17230336 ]
ASF GitHub Bot commented on GEODE-8700: --------------------------------------- gaussianrecurrence commented on a change in pull request #691: URL: https://github.com/apache/geode-native/pull/691#discussion_r521793026 ########## File path: cppcache/include/geode/AttributesMutator.hpp ########## @@ -190,6 +191,22 @@ class APACHE_GEODE_EXPORT AttributesMutator { */ void setCacheWriter(const std::string& libpath, const std::string& factoryFuncName); + + /** Sets partition resolver writer for region. The previous partition resolver + * will be replaced with <code>resolver</code>. + * @param resolver PartitionResolver + */ + void setPartitionResolver(std::shared_ptr<PartitionResolver> resolver); + + /** Sets cache writer for region. The previous partition resolver will be + * replaced with the one created using the factory function provided in + * the given library. + * @param library_path path of the library containing partition resolver + * factory function. + * @param function factory function for creating a partition resolver. + */ + void setPartitionResolver(const std::string& library_path, Review comment: Indeed, I don't actually need the library one. Good catch :) will remove it! ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Add the ability to change the partition resolver > ------------------------------------------------ > > Key: GEODE-8700 > URL: https://issues.apache.org/jira/browse/GEODE-8700 > Project: Geode > Issue Type: New Feature > Components: native client > Affects Versions: 1.13.0 > Reporter: Mario Salazar de Torres > Assignee: Mario Salazar de Torres > Priority: Major > Labels: pull-request-available > > *AS A* native client contributor > *I WANT* to be able to mutate partition resolver > *SO THAT* I don't need to create a shared library to specify one. -- This message was sent by Atlassian Jira (v8.3.4#803005)