-
Notifications
You must be signed in to change notification settings - Fork 288
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
[Bug][Seatunnel-web] Task instance filter is not working as expected #204
Conversation
bc7b067
to
2d672e8
Compare
Please fix ci. |
Seems PR CI is failed because of CI environment issue Let me retrigger the PR CI by pushing dummy change. |
2d672e8
to
6bb583b
Compare
Error: /home/runner/work/seatunnel-web/seatunnel-web/seatunnel-server/seatunnel-server-common/src/main/java/org/apache/seatunnel/server/common/CodeGenerateUtils.java:[22,12] cannot access java.util |
…o taskName as taskName is referred everywhere in UI.
6bb583b
to
b3c2a90
Compare
Finally build is success. |
@@ -105,7 +105,7 @@ | |||
<hadoop-aws.version>3.1.4</hadoop-aws.version> | |||
<aws-java-sdk-bundle.version>1.11.271</aws-java-sdk-bundle.version> | |||
<spotless.version>2.29.0</spotless.version> | |||
<logback.version>1.5.7</logback.version> | |||
<logback.version>1.2.3</logback.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why we should downgrade logback version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Despite upgrading to version 1.5.7, there remains an unresolved Common Vulnerabilities and Exposures (CVE) issue in this version.
- Upon upgrading to 1.5.7, the log4j-slf4j-impl jar file is absent from the project, resulting in failed logging attempts.
So reverted as of now. Will check more on this later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks @arshadmohammad
Thanks @Hisoka-X for reviewing and merging the PR. |
Corredcted task instance filter logic
Changed logback version to 1.2.3
Purpose of this pull request
Check list
New License Guide