[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-12 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16396401#comment-16396401 ] Vihang Karajgaonkar commented on HIVE-17580: patch merged to standalone-metastore branch.

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-10 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16394361#comment-16394361 ] Vihang Karajgaonkar commented on HIVE-17580: Fixed TestAvroObjectInspectorGenerator. Rest of

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-10 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16394170#comment-16394170 ] Hive QA commented on HIVE-17580: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-09 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16393906#comment-16393906 ] Vihang Karajgaonkar commented on HIVE-17580: Thanks for the review [~alangates] Attached the

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-08 Thread Alan Gates (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16392082#comment-16392082 ] Alan Gates commented on HIVE-17580: --- +1 for the latest PR.  Thank Vihang, this was a lot of work. >

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-08 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16392022#comment-16392022 ] Vihang Karajgaonkar commented on HIVE-17580: I update the PR to include the above proposed

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-06 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388946#comment-16388946 ] Vihang Karajgaonkar commented on HIVE-17580: Thanks [~owen.omalley] for explaining. In that

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-06 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388658#comment-16388658 ] Owen O'Malley commented on HIVE-17580: -- {quote}I don't see a particular logically grouping of classes

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-06 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388655#comment-16388655 ] Owen O'Malley commented on HIVE-17580: -- {quote}The other option suggested of moving ObjectInspector

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-05 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16387354#comment-16387354 ] Vihang Karajgaonkar commented on HIVE-17580: Combining the enums is not going to help solve

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-05 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386564#comment-16386564 ] Owen O'Malley commented on HIVE-17580: -- The problem with putting ObjectInspector into storage-api is

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-03-01 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16382432#comment-16382432 ] Vihang Karajgaonkar commented on HIVE-17580: [~owen.omalley] [~alangates] any thoughts on my

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-28 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16381165#comment-16381165 ] Vihang Karajgaonkar commented on HIVE-17580: on a side note, took a look at the test failures

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-28 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16381124#comment-16381124 ] Vihang Karajgaonkar commented on HIVE-17580: Thanks [~owen.omalley] for taking a look. bq.

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-28 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16381021#comment-16381021 ] Hive QA commented on HIVE-17580: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-28 Thread Owen O'Malley (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16380792#comment-16380792 ] Owen O'Malley commented on HIVE-17580: -- I'm not happy about ObjectInspector moving into storage-api.

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-28 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16380769#comment-16380769 ] Hive QA commented on HIVE-17580: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-28 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16380695#comment-16380695 ] Vihang Karajgaonkar commented on HIVE-17580: resubmitted patch to trigger precommit. 092 is

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-27 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16379860#comment-16379860 ] Vihang Karajgaonkar commented on HIVE-17580: latest patch restores {{Category}} enum back into

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-26 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16376509#comment-16376509 ] Vihang Karajgaonkar commented on HIVE-17580: I tried to run TestTablesCreateDropAlterTruncate

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-25 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16376197#comment-16376197 ] Hive QA commented on HIVE-17580: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-25 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16376164#comment-16376164 ] Vihang Karajgaonkar commented on HIVE-17580: added {{mavenTestArgs = -Dtest.groups=""}} to

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-24 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16375930#comment-16375930 ] Hive QA commented on HIVE-17580: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-24 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16375907#comment-16375907 ] Vihang Karajgaonkar commented on HIVE-17580: Most of these failures are because

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-24 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16375663#comment-16375663 ] Hive QA commented on HIVE-17580: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-23 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16375360#comment-16375360 ] Vihang Karajgaonkar commented on HIVE-17580: Posting the pre-commit result here. 294 tests

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-23 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16374823#comment-16374823 ] Vihang Karajgaonkar commented on HIVE-17580: pre-commit didn't trigger for some reason.

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-22 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16373700#comment-16373700 ] Vihang Karajgaonkar commented on HIVE-17580: Hi [~alangates] the latest patch attached here

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-02-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16373685#comment-16373685 ] ASF GitHub Bot commented on HIVE-17580: --- GitHub user vihangk1 opened a pull request:

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-01-22 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335179#comment-16335179 ] Vihang Karajgaonkar commented on HIVE-17580: Added the fix for the failing tests. Updated the

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-01-22 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16334582#comment-16334582 ] Vihang Karajgaonkar commented on HIVE-17580: Test failures look related. I will take a look.

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-01-22 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16334052#comment-16334052 ] Hive QA commented on HIVE-17580: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-01-21 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16333955#comment-16333955 ] ASF GitHub Bot commented on HIVE-17580: --- GitHub user vihangk1 opened a pull request:

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-01-11 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16323292#comment-16323292 ] Vihang Karajgaonkar commented on HIVE-17580: The reason why most of the tests are failing is

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-01-09 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16318918#comment-16318918 ] Vihang Karajgaonkar commented on HIVE-17580: [~alangates] I think its failing because its not

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-01-09 Thread Alan Gates (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16318831#comment-16318831 ] Alan Gates commented on HIVE-17580: --- I suspect HiveQA is getting tripped up by 'standalone-metastore'

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-01-09 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16318824#comment-16318824 ] Vihang Karajgaonkar commented on HIVE-17580: HiveQA does not seem to like patches on

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-01-05 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16314314#comment-16314314 ] Vihang Karajgaonkar commented on HIVE-17580: [~alangates] [~owen.omalley] and [~sershe] Can

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2018-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16314309#comment-16314309 ] ASF GitHub Bot commented on HIVE-17580: --- GitHub user vihangk1 opened a pull request:

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2017-12-03 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16275877#comment-16275877 ] Vihang Karajgaonkar commented on HIVE-17580: [~sershe], [~alangates], [~owen.omalley],

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2017-11-28 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16270239#comment-16270239 ] Vihang Karajgaonkar commented on HIVE-17580: After moving Deserializer, Serializer,

[jira] [Commented] (HIVE-17580) Remove dependency of get_fields_with_environment_context API to serde

2017-09-25 Thread Alan Gates (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-17580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16179431#comment-16179431 ] Alan Gates commented on HIVE-17580: --- I'm +1 on using the SARGs for this, but I think we will have to