[GitHub] spark issue #17268: [SPARK-19932][SS] Disallow a case that might cause OOM f...

[marmbrus <gi...@git.apache.org>]

Github user marmbrus commented on the issue:

    https://github.com/apache/spark/pull/17268
  
    Sorry I'm still not sure if this is a good idea.
    
    Why disallow the following,
    
    ```scala
    spark
       .readStream             
       .withWatermark("eventTime", "10 seconds")
       .dropDuplicates("id")
    ```
    
    But not this query?
    
    ```scala
    spark
       .readStream             
       .withWatermark("eventTime", "10 seconds")
       .groupBy("id")
       .agg(first(struct($"*")))
    ```
    
    I think its an incredibly difficult problem to tell people if their query is scalable without seeing the data.  I'm not sure if we want to do it piecemeal as I'm concerned that is more confusing.


---
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