[jira] [Commented] (AURORA-1963) Structdump endpoint shows thrift metadata

["Bill Farner (JIRA)" <ji...@apache.org>]

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

Bill Farner commented on AURORA-1963:
-------------------------------------

{quote}As the structdump is the only way to get the full configuration of a task{quote}

This is untrue.  A task's configuration may be fetched via a {{getTasksStatus}} call to the thrift API, which is the supported route.

> Structdump endpoint shows thrift metadata
> -----------------------------------------
>
>                 Key: AURORA-1963
>                 URL: https://issues.apache.org/jira/browse/AURORA-1963
>             Project: Aurora
>          Issue Type: Bug
>          Components: Scheduler
>    Affects Versions: 0.19.0
>         Environment: Devcluster VM
>            Reporter: Juan Manuel Fresia
>            Priority: Minor
>              Labels: patch
>
> The structdump endpoint changed the way it displays the task config. This change made some thrift internal metadata visible, such as {{__isset_bitfield}} on optionals fields and appending underscores on the names of other fields.
> For example, here is a fragment of a 0.19 structdump output:
> {code}
>             "timestamp": 1514573549125,
>             "status": "RUNNING",
>             "scheduler": "ee5feaf3d8b1",
>             "__isset_bitfield": 1,
>             "optionals": [
>               "MESSAGE",
>               "SCHEDULER"
>             ]
> {code}
> And here is the same fragment, but with 0.18 scheduler:
> {code}
>         Item[2] = 
>           timestamp: 1514574966299
>           status: RUNNING
>           message: not set
>           scheduler: "84611e03fe9e"
> {code}
> As the structdump is the only way to get the full configuration of a task, it is inconvenient to have those fields and namings messing around.
> The issue seems to be caused by a change in the way data taken from thrift is converted to be pretty printed; by changing a thrift specific converter to a GSON serializer. The proposed fix is to backport the old converter and restore the old structdump style.
> The review of the change related to the issue is [here|https://reviews.apache.org/r/63418/diff/3#16]
> The old pretty print utility used in 0.18 is  [here|https://github.com/apache/aurora/blob/87eb8914b3d79a0d27a91c50cc7070970e47abf9/commons/src/main/java/org/apache/aurora/common/thrift/Util.java]



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)