aurora-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Sirois (JIRA)" <>
Subject [jira] [Commented] (AURORA-1651) I* entity objects are lossy for primitive isSet-ness
Date Thu, 24 Mar 2016 17:27:25 GMT


John Sirois commented on AURORA-1651:

Would you object to ripping off the bandaid and killing {{isSet}} methods from the {{I*} generated
classes completely?  This would eliminate the confusing difference and make it clear in a
refactor from thrift struct to {{I*}} that you needed to handle the {{isSet}} in some other
way alltogether.

> I* entity objects are lossy for primitive isSet-ness
> ----------------------------------------------------
>                 Key: AURORA-1651
>                 URL:
>             Project: Aurora
>          Issue Type: Bug
>          Components: Build, Scheduler
>            Reporter: John Sirois
> For example, {{ITaskQuery}} sets the {{offset}} and {{limit}} like so:
> {noformat}
>     this.offset = wrapped.getOffset();
>     this.limit = wrapped.getLimit();
> {noformat}
> There is no check to see if {{wrapped.isSetOffset}} or {{wrapped.isSetLimit}} paired
with use of a nullable wrapper type ({{Integer}} vs {{int}} in this case) or an {{isSet}}
bit vector.  As a result a round trip from thrift struct to entity to thrift struct loses

This message was sent by Atlassian JIRA

View raw message