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

add ignore *.log for hs_err_pid*.log/replay_pid*.log #5275

Closed
wants to merge 1 commit into from

Conversation

qxo
Copy link
Contributor

@qxo qxo commented Aug 8, 2020

Please answer these questions before submitting a pull request

  • Why submit this pull request?

  • Bug fix

  • New feature provided

  • Improve performance

  • Related issues


Bug fix

  • Bug description.

  • How to fix?


New feature or improvement

  • Describe the details and related test reports.

@wu-sheng
Copy link
Member

wu-sheng commented Aug 9, 2020

This pr conflicts with master. And where are these logs from?

hs_err_pid*.log/replay_pid*.log generated when jvm crash on maven build.
@qxo
Copy link
Contributor Author

qxo commented Aug 9, 2020

This pr conflicts with master. And where are these logs from?

hs_err_pid*.log/replay_pid*.log generated when jvm crash on maven build.

@wu-sheng
Copy link
Member

wu-sheng commented Aug 9, 2020

That is unexpected, why we need to ignore those? We don't have tests for JVM crashing. The ignore file is not for all caees, it just ignore for tool config or test generated files.

@wu-sheng wu-sheng closed this Aug 9, 2020
@wu-sheng wu-sheng added the wontfix This will not be worked on label Aug 9, 2020
@wu-sheng wu-sheng added this to the 8.2.0 milestone Aug 9, 2020
@qxo
Copy link
Contributor Author

qxo commented Aug 9, 2020

That is unexpected, why we need to ignore those? We don't have tests for JVM crashing. The ignore file is not for all caees, it just ignore for tool config or test generated files.

当构建时内存不足就会产生这些临时文件, we should ignore it just like target dir.

@wu-sheng
Copy link
Member

wu-sheng commented Aug 9, 2020

You could have many files, such as dump file. It is not reasonable to add ignore for all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants