[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16031808#comment-16031808
]
Sergio Peña commented on HIVE-16164:
[~sushanth] Thanks for the post notes. I will indeed require your
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16023816#comment-16023816
]
Alexander Kolbasov commented on HIVE-16164:
---
[~sushanth] The tricky part with Kafka messages
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16019049#comment-16019049
]
Sushanth Sowmyan commented on HIVE-16164:
-
Hi, just making a note here, post patch:
a) I like
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15990163#comment-15990163
]
Lefty Leverenz commented on HIVE-16164:
---
Okay, thanks Sergio.
> Provide mechanism for passing HMS
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15987111#comment-15987111
]
Sergio Peña commented on HIVE-16164:
Thanks [~leftylev]. I committed the patch to branch-2.3 as well.
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15980865#comment-15980865
]
Lefty Leverenz commented on HIVE-16164:
---
Does this need to be documented in the wiki?
> Provide
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15980864#comment-15980864
]
Lefty Leverenz commented on HIVE-16164:
---
[~spena], this went into branch-2 shortly after branch-2.3
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954463#comment-15954463
]
Hive QA commented on HIVE-16164:
Here are the results of testing the latest attachment:
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954302#comment-15954302
]
Mohit Sabharwal commented on HIVE-16164:
+1, LGTM pending tests
> Provide mechanism for passing
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15947875#comment-15947875
]
Hive QA commented on HIVE-16164:
Here are the results of testing the latest attachment:
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15941327#comment-15941327
]
Hive QA commented on HIVE-16164:
Here are the results of testing the latest attachment:
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15927141#comment-15927141
]
Alexander Kolbasov commented on HIVE-16164:
---
The generic case I can think of, for example is
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15926901#comment-15926901
]
Mohit Sabharwal commented on HIVE-16164:
One listener is called only on success, the other can get
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15926892#comment-15926892
]
Sergio Peña commented on HIVE-16164:
Agree. I've been thinking on this EnvironmentContext since I
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15926870#comment-15926870
]
Alexander Kolbasov commented on HIVE-16164:
---
Adding id to the ListenerEvent makes a perfect
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15926850#comment-15926850
]
Mohit Sabharwal commented on HIVE-16164:
I think using EnvironmentContext for writing internal
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15925637#comment-15925637
]
Hive QA commented on HIVE-16164:
Here are the results of testing the latest attachment:
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15925135#comment-15925135
]
Hive QA commented on HIVE-16164:
Here are the results of testing the latest attachment:
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15924893#comment-15924893
]
Sergio Peña commented on HIVE-16164:
Attached a patch for running all tests. The current tests that
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15905596#comment-15905596
]
Sergey Shelukhin commented on HIVE-16164:
-
Sorry, not familiar at all with the notification
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15905505#comment-15905505
]
Alexander Kolbasov commented on HIVE-16164:
---
I see. IMO a better way would be to just pass the
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15905294#comment-15905294
]
Sergio Peña commented on HIVE-16164:
The only reason I see is if the event was successful or not. I
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15904294#comment-15904294
]
Alexander Kolbasov commented on HIVE-16164:
---
I am also wondering why transactional listeners and
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15904284#comment-15904284
]
Alexander Kolbasov commented on HIVE-16164:
---
Note that in the comment snippet above the
[
https://issues.apache.org/jira/browse/HIVE-16164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15904126#comment-15904126
]
Sergio Peña commented on HIVE-16164:
[~sershe] Have you worked with the DB notification listener? If
25 matches
Mail list logo