Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SPARK-1559 #471

Closed
wants to merge 1 commit into from
Closed

SPARK-1559 #471

wants to merge 1 commit into from

Conversation

chu11
Copy link
Contributor

@chu11 chu11 commented Apr 21, 2014

Add conf dir to CLASSPATH dependent on whether SPARK_CONF_DIR is set or not.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@rxin
Copy link
Contributor

rxin commented Apr 24, 2014

This technically changes some behavior, but it is what I'd expect. @pwendell / @mateiz thoughts?

@andrewor14
Copy link
Contributor

Hi @chu11, the changes here are incorporated in #997. Would you mind closing this PR?

@chu11 chu11 closed this Sep 3, 2014
j-esse pushed a commit to j-esse/spark that referenced this pull request Jan 24, 2019
### Upstream SPARK-XXXXX ticket and PR link (if not applicable, explain)
No upstream PR - SafeLogging is palantir specific.

### What changes were proposed in this pull request?
If the log in SafeLogging is not marked as transient, it is included in the serialized version of TorrentBroadcast, which causes issues. This excludes the log from being included in the TorrentBroadcast object, by making the log transient.

### How was this patch tested?
No testing for this - the logic is taken from upstream
bzhaoopenstack pushed a commit to bzhaoopenstack/spark that referenced this pull request Sep 11, 2019
RolatZhang pushed a commit to RolatZhang/spark that referenced this pull request Aug 15, 2022
* KE-36501 fix replace function when first argument length equals second argument length

* KE-36499 fix replace function when  second argument is "" pushdown error

* KE-36494 fix replace function when second argument has special symbol "$", ")"

* fix spark version

* fix ut
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants