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.

Sign up for GitHub

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

Jump to bottom

config resource.storage.type=hdfs throw exception need config resource.storage.type=HDFS #3165

Closed
felix-thinkingdata opened this issue Jul 8, 2020 · 2 comments · Fixed by #3166
Closed

config resource.storage.type=hdfs throw exception need config resource.storage.type=HDFS #3165

felix-thinkingdata opened this issue Jul 8, 2020 · 2 comments · Fixed by #3166
Labels
bug Something isn't working

Comments

@felix-thinkingdata
Copy link
Contributor

when i config common.properties ,
resource.storage.type=hdfs

will throw exception

image

because Enumerated type comparisons do not ignore case。
@felix-thinkingdata felix-thinkingdata added the bug Something isn't working label Jul 8, 2020
felix-thinkingdata added a commit to felix-thinkingdata/incubator-dolphinscheduler that referenced this issue Jul 8, 2020
@felix-thinkingdata
fix bug apache#3165 get resource.storage.type value toUpperCase
79d51ff
@felix-thinkingdata felix-thinkingdata mentioned this issue Jul 8, 2020
Merged
@lgcareer
Copy link
Contributor

lgcareer commented Jul 9, 2020 •
edited
Loading

image

hi,The resourceStorageType parameter of install_config.conf must be one of the values of HDFS,S3,NONE.

@davidzollo davidzollo linked a pull request Jul 9, 2020 that will close this issue
fix bug #3165 get resource.storage.type value toUpperCase #3166
Merged
davidzollo added a commit that referenced this issue Jul 9, 2020
@felix-thinkingdata @davidzollo
fix bug #3165 get resource.storage.type value toUpperCase (#3166)
a23a3e2 
Co-authored-by: dailidong <dailidong66@gmail.com>
lgcareer pushed a commit that referenced this issue Jul 10, 2020
@felix-thinkingdata @davidzollo
#3176 optimize Gets the value of this property “resource.storage.type…
1e7582e 
…”， Comparison with enumerated types (#3178)

* fix bug #3165   get resource.storage.type value   toUpperCase

* fix bug #3176  optimize Gets the value of this property “resource.storage.type”

Co-authored-by: dailidong <dailidong66@gmail.com>
@felix-thinkingdata
Copy link
Contributor Author

felix-thinkingdata commented Jul 10, 2020 •
edited
Loading

image

hi,The resourceStorageType parameter of install_config.conf must be one of the values of HDFS,S3,NONE.

My expectation is to make mistakes less likely.
image

qiaozhanwei pushed a commit that referenced this issue Aug 10, 2020
@felix-thinkingdata @davidzollo
[Fix-3262][common] When you request the URL through applicationID to …
5584f0c 
…get the application status, you cannot get it if Kerberos authentication is enabled (#3264)

* fix bug #3165   get resource.storage.type value   toUpperCase

* fix bug #3176  optimize Gets the value of this property “resource.storage.type”

* fix When you request the URL through applicationID to get the application status, you cannot get it if Kerberos authentication is enabled

* fix When you request the URL through applicationID to get the application status, you cannot get it if Kerberos authentication is enabled

* fix When you request the URL through applicationID to get the application status, you cannot get it if Kerberos authentication is enabled

* fix miss

* add KerberosHttpClient fix  kerberos bug

* fix map init

* add juint test

* Extraction of public methods

* Extraction of public methods

* Fix code quality

* Fix code quality

* Fix code quality

* Fix code quality

* Fix code quality

* Fix code quality

* Fix code quality

* rebuild test

* rebuild test

* check style

* check style

* check style

* revert UT POM

* Kerberos judgment goes up to hadoopUtils

* fix merge

* Remove connection pool shutdown

Co-authored-by: dailidong <dailidong66@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
2 participants
@lgcareer @felix-thinkingdata