spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zsxwing <...@git.apache.org>
Subject [GitHub] spark pull request: [SPARK-6862][Streaming][WebUI][WIP] Add BatchP...
Date Tue, 14 Apr 2015 05:40:30 GMT
Github user zsxwing commented on the pull request:

    https://github.com/apache/spark/pull/5473#issuecomment-92613622
  
    > There should be tests in the StreamingListenerSuite that tests the local property
setting.
    
    As I explained in `https://github.com/apache/spark/pull/5288#issuecomment-92579899`, we
may get wrong properties in `SparkListenerJobStart.properties`. Because the tests will be
non-deterministic, I think it's better to add them after @JoshRosen fixes the issue.
    
    > In the screenshots, is there any way to limit the max col size of the error? When
there is error, it squishes all the other columns.
    
    Let me try some css style to fix it.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


Mime
View raw message