pirk-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PIRK-79) Allow json serialization to deserialize a queryInfo object that has only the name of a query schema.
Date Mon, 31 Oct 2016 03:48:58 GMT

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

ASF GitHub Bot commented on PIRK-79:
------------------------------------

Github user smarthi commented on a diff in the pull request:

    https://github.com/apache/incubator-pirk/pull/115#discussion_r85682663
  
    --- Diff: src/main/java/org/apache/pirk/response/wideskies/Response.java ---
    @@ -71,4 +71,29 @@ public void addElement(int position, BigInteger element)
       {
         responseElements.put(position, element);
       }
    +
    +  @Override public boolean equals(Object o)
    +  {
    +    if (this == o)
    +      return true;
    +    if (o == null || getClass() != o.getClass())
    +      return false;
    +
    +    Response response = (Response) o;
    +
    +    if (responseVersion != response.responseVersion)
    +      return false;
    +    if (!queryInfo.equals(response.queryInfo))
    +      return false;
    +    return responseElements.equals(response.responseElements);
    +
    --- End diff --
    
    Can we use Java 8 - Objects.equal() functionality for all equals() methods ?


> Allow json serialization to deserialize a queryInfo object that has only the name of
a query schema.
> ----------------------------------------------------------------------------------------------------
>
>                 Key: PIRK-79
>                 URL: https://issues.apache.org/jira/browse/PIRK-79
>             Project: PIRK
>          Issue Type: Improvement
>          Components: Querier
>            Reporter: Walter Ray-Dulany
>            Assignee: Walter Ray-Dulany
>             Fix For: 0.3.0
>
>
> In Pirk Java serialization, when deserializing a queryinfo object, one can get the requisite
query schema by deserializing an entire serialized query schema object *OR* by reading the
name of a query schema (stored in queryType in the QueryInfo object) and querying against
a registry of known queryschemas. Currently, JSON serialization does the former, but not the
latter.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message