Skip to content

[Fix-4578][admin] Fix PostgreSQL job model overview - #4579

Open
gordan-zhang wants to merge 1 commit into
DataLinkDC:devfrom
gordan-zhang:fix-4578-pg-job-overview
Open

gordan-zhang wants to merge 1 commit into
DataLinkDC:devfrom
gordan-zhang:fix-4578-pg-job-overview

Conversation

@gordan-zhang

Copy link
Copy Markdown

Purpose of the pull request

Fix the Home page batch/stream job overview returning zero counts when Dinky uses PostgreSQL as its metadata database.

Closes #4578.

Brief change log

  • Count batch_model with explicit boolean predicates that work with PostgreSQL and remain compatible with MySQL/H2.
  • Compare normalized lowercase dialect values so PostgreSQLPrepareInterceptor lowercasing the SQL does not break matches against stored values such as FlinkSql.
  • Add an explicit result map with snake-case SQL aliases for stable PostgreSQL-to-Java property mapping.

Verify this pull request

  • Parsed TaskMapper.xml successfully with an XML parser.
  • Ran git diff --check successfully.
  • Verified the mapper against Dinky 1.2.5 with PostgreSQL 18: the API changed from streamingJobCount: 0 to the expected streamingJobCount: 10, with batchJobCount: 0.
  • Confirmed two existing Flink streaming jobs remained RUNNING after the Dinky-only restart.

Co-authored-by: Cursor <cursoragent@cursor.com>

This branch has not been deployed

No deployments
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.

[Bug] Batch and stream job overview returns zero with PostgreSQL

1 participant