[#HIVE-593] UDF and GenericUDF clean-up

Hadoop Hive

UDF and GenericUDF clean-up

Created: 30/Jun/09 01:24 AM Updated: 02/Jul/09 11:47 PM

Return to search

Component/s:

None

Affects Version/s:

0.3.0, 0.3.1

Fix Version/s:

0.4.0

Time Tracking:

Not Specified

File Attachments:

File Name	Date Attached	Attached By	Size
HIVE-593.1.patch	2009-07-01 09:03 PM	Zheng Shao	58 kB
HIVE-593.2.patch	2009-07-02 06:03 PM	Zheng Shao	58 kB

Resolution Date:

02/Jul/09 11:47 PM

Description

« Hide

We should deprecate ExprNodeIndexEvaluator and ExprNodeFieldEvaluator and use GenericUDF.
We should also clean up ExprNodeFuncEvaluator - there are a lot of hacks to be removed (hacks to get map, list and struct to work)

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

Zheng Shao added a comment - 01/Jul/09 09:03 PM

This patch replaces exprNodeIndexDesc and exprNodeIndexEvaluator with GenericUDF "index".

The removal of exprNodeFieldDesc will require some additional work (mainly that we need to serialize the field name together with the GenericUDF, but currently, GenericUDF is not serializable). I think we should move that to a separate JIRA, in which we will make GenericUDF optional implement Serializable, so that we can store compile-time information for use at the runtime.

[ Permlink | « Hide ]

He Yongqiang added a comment - 02/Jul/09 12:56 PM

there is one error in line 97 of GenericUDFIndex, isWritable is not defined.

[ Permlink | « Hide ]

Zheng Shao added a comment - 02/Jul/09 06:03 PM

Updated with trunk and fixed the problem.

[ Permlink | « Hide ]

Prasad Chakka added a comment - 02/Jul/09 09:56 PM

looks good. submit once tests are passed.

[ Permlink | « Hide ]

Prasad Chakka added a comment - 02/Jul/09 11:47 PM

Thanks Zheng.