You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/10/25 20:04:41 UTC

[GitHub] [spark] ueshin opened a new pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'F401' comments

ueshin opened a new pull request #34385:
URL: https://github.com/apache/spark/pull/34385


   ### What changes were proposed in this pull request?
   
   Remove unnecessary 'F401' comments.
   
   ### Why are the changes needed?
   
   Now that `flake8` in Jenkins was upgraded, we can remove unnecessary 'F401' comments.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Existing tests.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] ueshin commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
ueshin commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952470453


   @shaneknapp Thanks for taking a look at this!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952394359


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49099/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951365240






-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951485554


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49070/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951266264


   **[Test build #144598 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144598/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952424085


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49100/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952366011


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144630/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952366011


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144630/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951357035


   Kubernetes integration test status failure
   URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49069/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952363221


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144629/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952365988


   **[Test build #144630 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144630/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).
    * This patch **fails Python style tests**.
    * This patch merges cleanly.
    * This patch adds no public classes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] ueshin commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
ueshin commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-1002275027


   Thanks! merging to master.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] shaneknapp commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
shaneknapp commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952348639


   test this please


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] HyukjinKwon removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
HyukjinKwon removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951428827


   Merged to master.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951431455


   **[Test build #144600 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144600/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] ueshin commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
ueshin commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951460112


   That's weird. Actually I tried it with `flake8==3.9.0` locally and it passes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] ueshin closed pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
ueshin closed pull request #34385:
URL: https://github.com/apache/spark/pull/34385


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] shaneknapp commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
shaneknapp commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952339898


   > +1 for the PR itself. I'm also not sure about the status of Jenkins.
   
   yeah, that's definitely weird.  i ran `lint-python` manually, as jenkins, on a worker, in an identical environment as the build w/a fresh clone of spark and it passed.  let's see what the build i just triggered did


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952340025


   **[Test build #144629 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144629/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951267269


   **[Test build #144598 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144598/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).
    * This patch **fails Python style tests**.
    * This patch merges cleanly.
    * This patch adds no public classes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] ueshin commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
ueshin commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951278564


   Jenkins, retest this please.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951284476


   **[Test build #144599 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144599/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] HyukjinKwon commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951428827


   Merged to master.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] dongjoon-hyun commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
dongjoon-hyun commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-1001834205


   +1 for merging, too.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951286475


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144599/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] ueshin edited a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
ueshin edited a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951460112


   That's weird. Actually I tried it with `flake8==3.9.0` locally and it passes.
   
   ```
   % flake8 --version
   3.9.0 (mccabe: 0.6.1, pycodestyle: 2.7.0, pyflakes: 2.3.1) CPython 3.9.6 on Darwin
   
   % ./dev/lint-python
   starting python compilation test...
   python compilation succeeded.
   
   starting black test...
   black checks passed.
   
   starting pycodestyle test...
   pycodestyle checks passed.
   
   starting flake8 test...
   flake8 checks passed.
   
   starting mypy test...
   mypy checks passed.
   
   
   all lint-python tests passed!
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] shaneknapp commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
shaneknapp commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952393997


   yeah, i'm confused as to why this is failing.  leave this open, and i'll look in to the underlying failure later this week/early next.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952415864


   Kubernetes integration test status failure
   URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49100/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951267300


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144598/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951286447


   **[Test build #144599 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144599/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).
    * This patch **fails Python style tests**.
    * This patch merges cleanly.
    * This patch adds no public classes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951284476


   **[Test build #144599 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144599/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951266264


   **[Test build #144598 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144598/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] shaneknapp commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
shaneknapp commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952337664


   test this please


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952386756


   Kubernetes integration test status failure
   URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49099/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] shaneknapp commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
shaneknapp commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-953104497


   > @shaneknapp Thanks for taking a look at this!
   
   i created a 2nd PR to do some deeper testing of this:  https://github.com/apache/spark/pull/34404


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951327691


   Kubernetes integration test starting
   URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49069/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951319694


   Kubernetes integration test starting
   URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49068/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951432188


   **[Test build #144600 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144600/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).
    * This patch **fails Python style tests**.
    * This patch merges cleanly.
    * This patch adds no public classes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] shaneknapp commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
shaneknapp commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-953137588


   i straced lint-python in the other PR and am preparing myself to lose a few brain cells digging through the output later today.  :)
   
   https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144666/console


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951350975


   Kubernetes integration test status failure
   URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49068/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951365238






-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952340025


   **[Test build #144629 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144629/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] ueshin commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
ueshin commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951420896


   Jenkins, retest this please.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952365298


   **[Test build #144630 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144630/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952424085


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49100/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951485554


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49070/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951478530


   Kubernetes integration test status failure
   URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49070/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951432207


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144600/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951431455


   **[Test build #144600 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144600/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951432207


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144600/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952394359


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49099/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952386506


   Kubernetes integration test starting
   URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49100/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951286475


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144599/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] HyukjinKwon commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951429157


   ```
   
   ./python/pyspark/pandas/_typing.py:25:5: F401 'pyspark.pandas.base.IndexOpsMixin' imported but unused
       from pyspark.pandas.base import IndexOpsMixin
       ^
   ./python/pyspark/pandas/_typing.py:27:5: F401 'pyspark.pandas.generic.Frame' imported but unused
       from pyspark.pandas.generic import Frame
       ^
   2     F401 'pyspark.pandas.base.IndexOpsMixin' imported but unused
   2
   ```
   I think this is because Jenkins users old version of flake8 .. we'll have to find the way to make it compatible with old versions too for now until we drop Jenkins (end of this year). 
   
   Otherwise LGTM


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951455213


   Kubernetes integration test starting
   URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49070/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952341741


   **[Test build #144629 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144629/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).
    * This patch **fails Python style tests**.
    * This patch merges cleanly.
    * This patch adds no public classes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-951267300


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144598/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952363544


   Kubernetes integration test starting
   URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49099/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] AmplabJenkins removed a comment on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952363221


   
   Refer to this link for build results (access rights to CI server needed): 
   https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/144629/
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] SparkQA commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
SparkQA commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-952365298


   **[Test build #144630 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/144630/testReport)** for PR 34385 at commit [`44da384`](https://github.com/apache/spark/commit/44da384c661c31d27462cd96634b0cfb3cc83ba7).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


[GitHub] [spark] HyukjinKwon commented on pull request #34385: [SPARK-37011][PYTHON] Remove unnecessary 'noqa: F401' comments

Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on pull request #34385:
URL: https://github.com/apache/spark/pull/34385#issuecomment-1001826373


   I think we can merge it now since Jenkins is dropped :-). @ueshin would you mind rebasing this Pr please?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org