Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35227176
@JoshRosen any updates on this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. To do so, please
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35126548
@JoshRosen, can you review this and merge? This closes SPARK-1087 along
with SPARK-972.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35126310
Merged build finished.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35126312
All automated tests passed.
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12723/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35124160
Merged build triggered.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35124161
Merged build started.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35123080
Merged build finished.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35123081
All automated tests passed.
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12721/
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35120729
Wait, what happened to foreachPartition, keys and repartition methods? Did
I just accidentally deleted them or have they been discontinued?
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35120691
Merged build triggered.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35120692
Merged build started.
Github user JoshRosen commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35117763
I may have given some bad suggestions earlier with using
`_extract_concise_traceback`; I think you may just want to use the regular
traceback module and just u
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35117581
@JoshRosen, I think the problem is with file names with spaces in them. In
that case, the line split is not working and returning the wrong value. Lets
try movi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35116879
Merged build finished.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35116880
One or more automated tests failed
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12720/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35114290
Merged build started.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35114289
Merged build triggered.
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35111896
Jenkins, test this please.
Github user JoshRosen commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35114200
Jenkins, this is ok to test.
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35058781
@JoshRosen, I think when the traceback function doesn't return the callsite
info, it simply returns "I'm lost!". Handled that case in this commit. Can you
ask J
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35058209
One or more automated tests failed
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12712/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35058208
Merged build finished.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35056796
Merged build triggered.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35056797
Merged build started.
Github user JoshRosen commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35056776
Jenkins, test this please.
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35054785
Jenkins, test this please.
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35054777
I think this is good to go. Can I ask Jenkins to test this?
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35028748
One or more automated tests failed
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12710/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35028747
Merged build finished.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35026001
Merged build started.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35026000
Merged build triggered.
Github user JoshRosen commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-35025658
Jenkins, test this please.
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34961462
Can one of the admins test and merge this?
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34850215
@JoshRosen
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34836413
I think for future improvements, it would be a good idea to move the
traceback code to a separate file and use that to return callsite. Please start
a JIRA tick
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34804481
I have used the _extract_concise_traceback to print the callsite of
existing SparkContext. I guess this is what you had in mind.
Github user JoshRosen commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34799232
You may be able to re-use the
[`_extract_concise_traceback`](https://github.com/apache/incubator-spark/pull/311/files#diff-d6fe2792e44f6babc94aabfefc8b9bceR43)
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34798126
Got it. I will work on it and report it back to you. I guess traceback
module will be useful here.
Github user JoshRosen commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34797494
I'd store the constructor's call site (filename + line number) as an
instance variable inside SparkContext, since we already have the
`SparkContext._active_spa
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34796972
I understand. Maybe we can use a global variable to store the call site
(line number?) and print it back along with the address. What do you think?
Github user JoshRosen commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34795439
In [SPARK-972](https://spark-project.atlassian.net/browse/SPARK-972), my
intent was to log the _call site_ of the original SparkContext construction to
help fi
Github user jyotiska commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34782466
This tries to solve
[SPARK-972](https://spark-project.atlassian.net/browse/SPARK-972)
GitHub user jyotiska reopened a pull request:
https://github.com/apache/incubator-spark/pull/581
Added extra description on ValueError when one Spark context already exists
I added extra description on the ValueError message, when more than one
Spark context already exists. Added th
Github user jyotiska closed the pull request at:
https://github.com/apache/incubator-spark/pull/581
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/581#issuecomment-34758328
Can one of the admins verify this patch?
GitHub user jyotiska opened a pull request:
https://github.com/apache/incubator-spark/pull/581
Added extra description on ValueError when one Spark context already exists
I added extra description on the ValueError message, when more than one
Spark context already exists. Added the
46 matches
Mail list logo