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

Roshan Naik commented on HIVE-15691:
------------------------------------

[~ekoifman] & [~kalyanhadoop]

1) I think the class needs only 2 constructors:
- StrictRegexWriter(String regex, HiveEndPoint endPoint)
- StrictRegexWriter(String regex, HiveEndPoint endPoint, HiveConf conf)

The 'connection' param should be eliminated.

2) I cannot say much about the correctness of createSerde() & encode() methods, 
as i have not worked with RegexSerDe. Would be good to know if this has been 
validated via a manual test  where the data streamed via this writer, was 
properly queryable from Hive.



> Create StrictRegexWriter to work with RegexSerializer for Flume Hive Sink
> -------------------------------------------------------------------------
>
>                 Key: HIVE-15691
>                 URL: https://issues.apache.org/jira/browse/HIVE-15691
>             Project: Hive
>          Issue Type: New Feature
>          Components: HCatalog, Transactions
>            Reporter: Kalyan
>            Assignee: Kalyan
>         Attachments: HIVE-15691.1.patch, HIVE-15691.patch, 
> HIVE-15691-updated.patch
>
>
> Create StrictRegexWriter to work with RegexSerializer for Flume Hive Sink.
> It is similar to StrictJsonWriter available in hive.
> Dependency is there in flume to commit.
> FLUME-3036 : Create a RegexSerializer for Hive Sink.
> Patch is available for Flume, Please verify the below link
> https://github.com/kalyanhadooptraining/flume/commit/1c651e81395404321f9964c8d9d2af6f4a2aaef9



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to