Skip to content

feat: add langfuse#442

Merged
zxuexingzhijie merged 10 commits intospring-ai-alibaba:mainfrom
zihenzzz:add-langfuse
Feb 26, 2026
Merged

feat: add langfuse#442
zxuexingzhijie merged 10 commits intospring-ai-alibaba:mainfrom
zihenzzz:add-langfuse

Conversation

@zihenzzz
Copy link
Copy Markdown
Contributor

Describe what this PR does / why we need it

Does this pull request fix one issue?

#279

Describe how you did it

Describe how to verify it

Special notes for reviews

27bc28f678e41a96220df573f3bd64e5 7ac69e4203246f03987ba3bc6ccf52df

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

这个类的位置放的不太对,可以放到service里面,就近原则


@Slf4j
@Configuration
public class OpenTelemetryConfig {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

这个configuration的前缀跟其余配置类保持一致吧

Copy link
Copy Markdown
Collaborator

@zxuexingzhijie zxuexingzhijie left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown
Collaborator

@zxuexingzhijie zxuexingzhijie left a comment

Choose a reason for hiding this comment

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

相关的文档说明也添加一下吧,感谢

@zxuexingzhijie zxuexingzhijie merged commit 8c24663 into spring-ai-alibaba:main Feb 26, 2026
7 checks passed
hiyizi pushed a commit to hiyizi/DataAgent that referenced this pull request Feb 27, 2026
* feat: add langfuse

* feat: add langfuse

* feat: add langfuse

* feat: add langfuse

* feat: add langfuse

* feat: add langfuse

* feat: add langfuse

* feat: add langfuse

* feat: add langfuse

* feat: add langfuse doc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants