incubator-blur-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aaron McCurry (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (BLUR-289) Table Properties are not being persisted to zookeeper
Date Sun, 27 Oct 2013 14:14:32 GMT

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

Aaron McCurry commented on BLUR-289:
------------------------------------

Yes that is the code.

About the watcher on the individual fields, we only watch the enabled field.  All the other
fields are just there for read purposes.  And actually we could changes the watchers for the
enabled child node to just watch the table node for data changes.  Then we wouldn't need any
child nodes.

I think modifying of the watchers is secondary task.

And for the conversion, when we read the table descriptor into memory we should first check
to see if the data is null.  If it is, then it needs to be converted.  If not try to read
the bytes with the TJsonProto thrift code, if that fails throw an exception.  Otherwise it's
already in the correct format.

What do you think?

Aaron

> Table Properties are not being persisted to zookeeper
> -----------------------------------------------------
>
>                 Key: BLUR-289
>                 URL: https://issues.apache.org/jira/browse/BLUR-289
>             Project: Apache Blur
>          Issue Type: Bug
>          Components: Blur
>    Affects Versions: 0.3.0, 0.2.1
>            Reporter: Rahul Challapalli
>            Priority: Blocker
>             Fix For: 0.2.1
>
>         Attachments: persist_tableproperties.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message