Skip to content

Conversation

@hawk9821
Copy link
Contributor

Purpose of this pull request

LLM transforms Support custom field name

Does this PR introduce any user-facing change?

no

How was this patch tested?

E2E: TestLLMIT#testLLMWithOpenAIOutputColumnName

Check list

@hawk9821 hawk9821 force-pushed the llm_custom_inferenceName branch from 8088d05 to 4dae15c Compare September 12, 2024 16:17
@hawk9821 hawk9821 force-pushed the llm_custom_inferenceName branch from 5793222 to 1ea211d Compare September 13, 2024 14:55
Copy link
Member

@Hisoka-X Hisoka-X left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @hawk9821 !

@hailin0 hailin0 merged commit 6da7491 into apache:dev Sep 14, 2024
@hawk9821 hawk9821 deleted the llm_custom_inferenceName branch September 19, 2024 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants