Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[lineage] Introduce jdbc lineage meta implementations #2267

Closed

Conversation

FangYongs
Copy link
Contributor

Purpose

Linked issue: close #1480

This PR aims to introduce jdbc lineage meta with following capabilities:

  1. Configure jdbc relevant options such as url, user and password
  2. Create source/sink lineage tables if jdbc.auto-ddl is true

Tests

  1. Added test JdbcLineageMetaTest to validate auto ddl for source/sink lineage tables.

API and Format

no

Documentation

no

@FangYongs FangYongs force-pushed the jdbc_lineage_meta_interface branch from de38dc2 to 73706cf Compare November 6, 2023 08:24
Comment on lines +84 to +86
.replace(
EncodingUtils.escapeIdentifier(dataField.name()),
dataField.name())
Copy link
Contributor

Choose a reason for hiding this comment

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

Why does it need to be replaced?

"Jdbc password for Paimon to store some information such as lineage and meta.");

public static final ConfigOption<Boolean> JDBC_AUTO_DDL =
key("jdbc.auto-ddl")
Copy link
Contributor

Choose a reason for hiding this comment

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

How about replacing it with jdbc.auto-create?

Comment on lines +101 to +122
public static final ConfigOption<String> JDBC_URL =
key("jdbc.url")
.stringType()
.noDefaultValue()
.withDescription(
"Jdbc url for Paimon to store some information such as lineage and meta.");

public static final ConfigOption<String> JDBC_USER =
key("jdbc.user")
.stringType()
.noDefaultValue()
.withDescription(
"Jdbc user name for Paimon to store some information such as lineage and meta.");

public static final ConfigOption<String> JDBC_PASSWORD =
key("jdbc.password")
.stringType()
.noDefaultValue()
.withDescription(
"Jdbc password for Paimon to store some information such as lineage and meta.");

public static final ConfigOption<Boolean> JDBC_AUTO_DDL =
Copy link
Contributor

Choose a reason for hiding this comment

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

If these options are only provided to lineage for use, I prefer to add some prefixes in front, and considering other implementations in the future, the prefix can be lineage.kind.${identifier}.xxx.

@gmdfalk
Copy link

gmdfalk commented Oct 29, 2024

@FangYongs hey, this has been open for a while. are there still plans to merge it, maybe for the upcoming Paimon 1.0?

@JingsongLi
Copy link
Contributor

@FangYongs hey, this has been open for a while. are there still plans to merge it, maybe for the upcoming Paimon 1.0?

This is a unfinished issue... We should re-design this...

I close this now, feel free to re-open this if you want to drive this again.

@JingsongLi JingsongLi closed this Nov 28, 2024
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.

[Feature] Support jdbc lineage meta implementations
4 participants