Skip to content

put 'curRequestSize = 0' after 'logDebug' it#1477

Closed
JerryLead wants to merge 1 commit intoapache:masterfrom
JerryLead:patch-1
Closed

put 'curRequestSize = 0' after 'logDebug' it#1477
JerryLead wants to merge 1 commit intoapache:masterfrom
JerryLead:patch-1

Conversation

@JerryLead
Copy link
Contributor

This is a minor change. We should first logDebug($curRequestSize) and then set it to 0.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@rxin
Copy link
Contributor

rxin commented Jul 18, 2014

Jenkins, test this please.

1 similar comment
@rxin
Copy link
Contributor

rxin commented Jul 19, 2014

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Jul 19, 2014

QA tests have started for PR 1477. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16842/consoleFull

@SparkQA
Copy link

SparkQA commented Jul 19, 2014

QA results for PR 1477:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16842/consoleFull

@rxin
Copy link
Contributor

rxin commented Jul 19, 2014

Thanks. Merging in master.

@asfgit asfgit closed this in 805f329 Jul 19, 2014
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
This is a minor change. We should first logDebug($curRequestSize) and then set it to 0.

Author: Lijie Xu <csxulijie@gmail.com>

Closes apache#1477 from JerryLead/patch-1 and squashes the following commits:

aed722d [Lijie Xu] put 'curRequestSize = 0' after 'logDebug' it
sunchao pushed a commit to sunchao/spark that referenced this pull request Jun 2, 2023
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