Github user vanzin commented on the pull request:

    https://github.com/apache/spark/pull/2309#issuecomment-54875145
  
    > My concern is If it's handled in the Utils class we could have a case 
where the user is setting memory from the command line but is getting 
unexpected errors from the mislabeled environment variable.
    
    If that indeed happens, the code should be changed so that only the actual 
value being used is parsed. (The error, though, might be more benefitial than 
harmful: it tells the user there is something wrong somewhere in his 
configuration.)
    
    But well, let's see if any committers have a different opinion.


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to