stoty commented on code in PR #5870: URL: https://github.com/apache/hbase/pull/5870#discussion_r1590543304
########## hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/model/CellModel.java: ########## @@ -58,10 +61,11 @@ * </pre> */ @XmlRootElement(name = "Cell") -@XmlAccessorType(XmlAccessType.FIELD) +@XmlAccessorType(XmlAccessType.NONE) Review Comment: XmlAccessType.FIELD implicitly adds every field of the method to the XML. So if we kept XmlAccessType.FIELD, we'd have to annotate the transient fields. Since we are are already annotating every field that we wanto to marshall, it's simpler to just change the class-level setting. -- 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. To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
