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

增加运行参数use-lower-case 使用小写字母 #214

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

stevqin
Copy link

@stevqin stevqin commented Oct 23, 2023

--use-lower-case true
让生成的Doris表名与字段名使用小写

Proposed changes

Issue Number: close #xxx

Problem Summary:

Describe the overview of changes.

Checklist(Required)

  1. Does it affect the original behavior: (Yes/No/I Don't know)
  2. Has unit tests been added: (Yes/No/No Need)
  3. Has document been added or modified: (Yes/No/No Need)
  4. Does it need to update dependencies: (Yes/No)
  5. Are there any changes that cannot be rolled back: (Yes/No)

Further comments

If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...

--use-lower-case true
让生成的Doris表名与字段名使用小写
@DongLiang-0
Copy link
Contributor

Thanks for your contribution.
In doris you can set the lower_case_table_names variable to ignore case, so providing this functionality in a sync tool is a bit redundant.

@stevqin
Copy link
Author

stevqin commented Oct 23, 2023

Thanks for your contribution. In doris you can set the lower_case_table_names variable to ignore case, so providing this functionality in a sync tool is a bit redundant.

这个可能和 Doris中的 lower_case_table_names 变量不一样, 在 Doris中, 根据官方文档解释, 这个变量仅对Oralce数据库生效, 并且一旦对Oracle数据库生效后, 则会出现无法查询 Oralce中 小写字母的表.

lower_case_table_names

@DongLiang-0
Copy link
Contributor

Is it possible to default to lowercase when writing, and then turn on this switch when uppercase is needed?

@DongLiang-0
Copy link
Contributor

Since the community is global and there are many foreign developers, thank you for changing all descriptions to English.

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