[
https://issues.apache.org/jira/browse/HIVE-4914?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13776498#comment-13776498
]
Phabricator commented on HIVE-4914:
-----------------------------------
ashutoshc has requested changes to the revision "HIVE-4914 [jira] filtering via
partition name should be done inside metastore server (implementation)".
INLINE COMMENTS
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java:21 Why do we need
to this? Simply use runtimeSerializationKryo.
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java:68 Did you
encounter into any issue while serializing timestamps? We have other tests
which serialize timestamp via Kryo, we haven't encountered any problems there
yet.
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java:32
I don't see any error handling for method not found and failure to deserialize
expression exceptions. Are you planning to add them?
REVISION DETAIL
https://reviews.facebook.net/D12561
To: JIRA, ashutoshc, sershe
> filtering via partition name should be done inside metastore server
> (implementation)
> ------------------------------------------------------------------------------------
>
> Key: HIVE-4914
> URL: https://issues.apache.org/jira/browse/HIVE-4914
> Project: Hive
> Issue Type: Improvement
> Components: Metastore
> Reporter: Sergey Shelukhin
> Assignee: Sergey Shelukhin
> Attachments: D12561.5.patch, D12561.6.patch, HIVE-4914.01.patch,
> HIVE-4914.02.patch, HIVE-4914.03.patch, HIVE-4914.04.patch,
> HIVE-4914.05.patch, HIVE-4914.06.patch, HIVE-4914.D12561.1.patch,
> HIVE-4914.D12561.2.patch, HIVE-4914.D12561.3.patch, HIVE-4914.D12561.4.patch,
> HIVE-4914.D12645.1.patch, HIVE-4914-only-no-gen.patch, HIVE-4914-only.patch,
> HIVE-4914.patch, HIVE-4914.patch, HIVE-4914.patch
>
>
> Currently, if the filter pushdown is impossible (which is most cases), the
> client gets all partition names from metastore, filters them, and asks for
> partitions by names for the filtered set.
> Metastore server code should do that instead; it should check if pushdown is
> possible and do it if so; otherwise it should do name-based filtering.
> Saves the roundtrip with all partition names from the server to client, and
> also removes the need to have pushdown viability checking on both sides.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
