[
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)
