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

Fix compilation error of YamlCodec in jdk1.8 #13791

Merged
merged 1 commit into from
Feb 26, 2024
Merged

Conversation

oxsean
Copy link
Collaborator

@oxsean oxsean commented Feb 26, 2024

What is the purpose of the change

Fix compilation error of: no suitable method found for loadAs(String,Class<CAP#1>)

[ERROR] dubbo\dubbo-remoting\dubbo-remoting-http12\src\main\java\org\apache\dubbo\remoting\http12\message\codec\YamlCodec.java:[64,41] error: no suitable method found for loadAs(String,Class<CAP#1>)
[ERROR]     method Yaml.<T#1>loadAs(Reader,Class<? super T#1>) is not applicable
[ERROR]       (cannot infer type-variable(s) T#1
[ERROR]         (argument mismatch; String cannot be converted to Reader))
[ERROR]     method Yaml.<T#2>loadAs(String,Class<? super T#2>) is not applicable
[ERROR]       (cannot infer type-variable(s) T#2
[ERROR]         (argument mismatch; Class<CAP#1> cannot be converted to Class<? super T#2>))
[ERROR]     method Yaml.<T#3>loadAs(InputStream,Class<? super T#3>) is not applicable
[ERROR]       (cannot infer type-variable(s) T#3
[ERROR]         (argument mismatch; String cannot be converted to InputStream))
[ERROR]   where T#1,T#2,T#3 are type-variables:
[ERROR]     T#1 extends Object declared in method <T#1>loadAs(Reader,Class<? super T#1>)
[ERROR]     T#2 extends Object declared in method <T#2>loadAs(String,Class<? super T#2>)
[ERROR]     T#3 extends Object declared in method <T#3>loadAs(InputStream,Class<? super T#3>)
[ERROR]   where CAP#1 is a fresh type-variable:
[ERROR]     CAP#1 extends Object from capture of ?

Copy link

sonarcloud bot commented Feb 26, 2024

@AlbumenJ AlbumenJ merged commit 0fa88a1 into apache:3.3 Feb 26, 2024
17 of 19 checks passed
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.

None yet

2 participants