[
https://issues.apache.org/jira/browse/HIVE-14011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15474501#comment-15474501
]
Mohit Sabharwal commented on HIVE-14011:
----------------------------------------
instance is private and only accessible via getInstance, so this it should
always default to org.apache.hive.hcatalog.messaging.json.JSONMessageFactory
which is the current behavior
LGTM, +1 pending tests
cc [~alangates] in case he has comments.
> MessageFactory is not pluggable
> -------------------------------
>
> Key: HIVE-14011
> URL: https://issues.apache.org/jira/browse/HIVE-14011
> Project: Hive
> Issue Type: Bug
> Reporter: Sravya Tirukkovalur
> Attachments: HIVE-14011.patch
>
>
> Property "hcatalog.message.factory.impl.json" is available to use a custom
> message factory implementation. Although it is not pluggable as
> MessageFatcory is hardcoded to use JSONMessageFactory.
> https://github.com/apache/hive/blob/26b5c7b56a4f28ce3eabc0207566cce46b29b558/hcatalog/server-extensions/src/main/java/org/apache/hive/hcatalog/messaging/MessageFactory.java#L39
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
