HyukjinKwon commented on pull request #29333:
URL: https://github.com/apache/spark/pull/29333#issuecomment-673479039
https://github.com/HyukjinKwon/spark/runs/980576471 should show the example
of this PR. Once this example shows the results correctly, this PR should be
good to go.
HyukjinKwon commented on pull request #29333:
URL: https://github.com/apache/spark/pull/29333#issuecomment-673477836
https://github.com/HyukjinKwon/spark/runs/980576471 should show the example
of this PR. Before this PR is merged, the test results would not be reported
because the
HyukjinKwon commented on pull request #29333:
URL: https://github.com/apache/spark/pull/29333#issuecomment-673426729
I elaborated it in the PR description how it works and why I had to fork two
plugins.
This is an automated
HyukjinKwon commented on pull request #29333:
URL: https://github.com/apache/spark/pull/29333#issuecomment-672569879
Okay, GitHub team suggested a scenario that it could work out of the box.
It's a bit complicated then I thought but I will try. Yes, it is a nice-to-have
but I think it is
HyukjinKwon commented on pull request #29333:
URL: https://github.com/apache/spark/pull/29333#issuecomment-668945164
The latest commit above
(https://github.com/apache/spark/pull/29333/commits/2688f21f1852b3e6a577fa9292985b346b9bdf6d)
contains the problem in terms of forked repos and PRs
HyukjinKwon commented on pull request #29333:
URL: https://github.com/apache/spark/pull/29333#issuecomment-668935709
I am making some changes to demonstrate the problem in terms of the fork and
PRs. Please ignore the changes made from now on. I will switch back from the
draft later when
HyukjinKwon commented on pull request #29333:
URL: https://github.com/apache/spark/pull/29333#issuecomment-668933522
Sure, let me take a closer look for that approach. In worst case, we might
have to drop this and go back to the original @viirya's approach at #29169.
HyukjinKwon commented on pull request #29333:
URL: https://github.com/apache/spark/pull/29333#issuecomment-668354063
adding @srowen, @gatorsmile, @cloud-fan as well to collect more feedback.
This is an automated message from
HyukjinKwon commented on pull request #29333:
URL: https://github.com/apache/spark/pull/29333#issuecomment-668353951
BTW, let me know if you guys have a preference in one of the options. I can
investigate fewer options in this case.
HyukjinKwon commented on pull request #29333:
URL: https://github.com/apache/spark/pull/29333#issuecomment-668348425
Just to share the current status,
In
[ScaCap/action-surefire-report](https://github.com/ScaCap/action-surefire-report)
plugin (and all other similar plugins), it
10 matches
Mail list logo