Skip to content

[FLINK-37781][docs] Add elasticsearch lookup join document - #125

Open
MOBIN-F wants to merge 1 commit into
apache:mainfrom
MOBIN-F:lookupJoin-Doc
Open

[FLINK-37781][docs] Add elasticsearch lookup join document#125
MOBIN-F wants to merge 1 commit into
apache:mainfrom
MOBIN-F:lookupJoin-Doc

Conversation

@MOBIN-F

@MOBIN-F MOBIN-F commented May 13, 2025

Copy link
Copy Markdown

#39 support elasticsearch lookup join, but the lack of relevant documents

@boring-cyborg

boring-cyborg Bot commented May 13, 2025

Copy link
Copy Markdown

Thanks for opening this pull request! Please check out our contributing guidelines. (https://flink.apache.org/contributing/how-to-contribute.html)

@MOBIN-F

MOBIN-F commented May 13, 2025

Copy link
Copy Markdown
Author

@liyubin117 @MartijnVisser PTAL

@MOBIN-F MOBIN-F changed the title [FLINK-37781][doc] Add elasticsearch lookup join document [FLINK-37781][docs] Add elasticsearch lookup join document May 13, 2025
@liyubin117

Copy link
Copy Markdown
Contributor

@MOBIN-F Thanks for your contribution, I will review in the late few days.

使用该配置时 "lookup.cache" 必须设置为 "PARTIAL”。</td>
</tr>
<tr>
<td><h5>lookup.partial-cache.caching-missing-key</h5></td>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should be 'lookup.partial-cache.cache-missing-key'

"lookup.cache" must be set to "PARTIAL" to use this option. </td>
</tr>
<tr>
<td><h5>lookup.partial-cache.caching-missing-key</h5></td>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should be 'lookup.partial-cache.cache-missing-key'

@reswqa

reswqa commented Jun 29, 2026

Copy link
Copy Markdown
Member

It's good to have documents for dim join . @MOBIN-F Could you pick this up again?

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