[jira] [Commented] (GEODE-7076) Pdx should not have group, groups

["ASF subversion and git services (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GEODE-7076?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16927575#comment-16927575 ] 

ASF subversion and git services commented on GEODE-7076:
--------------------------------------------------------

Commit eabefacd3a8f14af3da4bc6d859ef9d982be7f83 in geode's branch refs/heads/develop from Darrel Schneider
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=eabefac ]

GEODE-7076: remove "group" for pdx configuration (#4035)

Co-authored-by: Darrel Schneider <ds...@pivotal.io>

* moved RestfulEndpoint up to AbstractConfiguration
* moved CorrespondsWith to AbstractConfiguration

> Pdx should not have group, groups
> ---------------------------------
>
>                 Key: GEODE-7076
>                 URL: https://issues.apache.org/jira/browse/GEODE-7076
>             Project: Geode
>          Issue Type: Improvement
>          Components: management
>            Reporter: Gang Yan
>            Assignee: Darrel Schneider
>            Priority: Major
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> For REST API V2 for management, Swagger shows that the model for Pdx includes "group", "groups".
>  None of these should be specified when doing a POST Pdx. If you do specify group you get an error saying Pdx is only at the cluster level.
>  group and groups comes from CacheElement. Could the Pdx class override them and mark them with @JsonIgnore?
> The java-api exposes these on Pdx and it would be best if it didn't. If we did some refactoring so that Pdx did not have these methods on it them this would clean up the rest-api automatically.
> Objective:
>  # find a way to hide these attributes from Swagger docs.
>  # is there a way to hide "group"/"groups" from the java api as well
> The best solution might be to just have "group" and not "groups".



--
This message was sent by Atlassian Jira
(v8.3.2#803003)