Skip to content

Align JdbcChatMemoryProperties with other modules in Spring Boot #3010

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

Closed
wants to merge 1 commit into from

Conversation

quaff
Copy link
Contributor

@quaff quaff commented May 6, 2025

Such as BatchProperties.Jdbc and IntegrationProperties.Jdbc

  1. Change type of initialize-schema from boolean to DatabaseInitializationMode
  2. Allow to customize schema location

@linarkou
Copy link
Contributor

linarkou commented May 6, 2025

Can you please add username/password properties like inspring.sql.init?
Usually not all users have DDL permissions, so it would be great to customize user here.
See DataSourceInitializationConfiguration for reference

@quaff
Copy link
Contributor Author

quaff commented May 7, 2025

Can you please add username/password properties like inspring.sql.init? Usually not all users have DDL permissions, so it would be great to customize user here. See DataSourceInitializationConfiguration for reference

BatchProperties.Jdbc, IntegrationProperties.Jdbc, JdbcSessionProperties and QuartzProperties.Jdbc do not support username/password.

Such as `BatchProperties.Jdbc` and `IntegrationProperties.Jdbc`

1. Change type of `initialize-schema` from `boolean` to `DatabaseInitializationMode`
2. Allow to customize schema location

Signed-off-by: Yanming Zhou <zhouyanming@gmail.com>
@linarkou
Copy link
Contributor

linarkou commented May 7, 2025

Thank you! Yes, you right, it is not about alignment, it is just a feature. But I think it can be useful in some usecases.
Created #3018 for it

@quaff
Copy link
Contributor Author

quaff commented May 7, 2025

Close it as duplicate of #2662.

@quaff quaff closed this May 7, 2025
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