Github user gatorsmile commented on the issue:

    https://github.com/apache/spark/pull/15432
  
    Unfortunately, not all the things have a standard to follow. That is why I 
suggested you to do a research about it. Like Oracle, it does not have such a 
function in their SQL-function list: 
https://docs.oracle.com/cd/B19306_01/server.102/b14200/functions001.htm
    
    Since you are doing the change in `rand`, I think you can check whether the 
existing `rand` behaves as expected and adds the missing test cases if needed. 
This JIRA is just trying to cover an edge case of a seed number. Why not 
checking whether we appropriately handle all the cases? Then, we do not need to 
submit more small fixes for `rand`, right?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to