Skip to content

Conversation

@techaddict
Copy link
Contributor

Any comments are welcome

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for this one u can put all of them on a single line, i.e.

if (i < 4) 0 == prng42.nextInt(3) else 0 == prng43.nextInt(3)

@rxin
Copy link
Contributor

rxin commented Apr 24, 2014

lgtm otherwise. Thanks!

@techaddict
Copy link
Contributor Author

@rxin done 👍 😉

@rxin
Copy link
Contributor

rxin commented Apr 24, 2014

Thanks. I merged this.

@asfgit asfgit closed this in a03ac22 Apr 24, 2014
asfgit pushed a commit that referenced this pull request Apr 24, 2014
Any comments are welcome

Author: Sandeep <[email protected]>

Closes #531 from techaddict/stylefix-1 and squashes the following commits:

7492730 [Sandeep] Pass 4
98b2428 [Sandeep] fix rxin suggestions
b5e2e6f [Sandeep] Pass 3
05932d7 [Sandeep] fix if else styling 2
08690e5 [Sandeep] fix if else styling

(cherry picked from commit a03ac22)
Signed-off-by: Reynold Xin <[email protected]>
pdeyhim pushed a commit to pdeyhim/spark-1 that referenced this pull request Jun 25, 2014
Any comments are welcome

Author: Sandeep <[email protected]>

Closes apache#531 from techaddict/stylefix-1 and squashes the following commits:

7492730 [Sandeep] Pass 4
98b2428 [Sandeep] fix rxin suggestions
b5e2e6f [Sandeep] Pass 3
05932d7 [Sandeep] fix if else styling 2
08690e5 [Sandeep] fix if else styling
helenyugithub pushed a commit to helenyugithub/spark that referenced this pull request Aug 20, 2019
* [SPARK-27267][CORE] Update snappy to avoid error when decompressing empty serialized data (apache#531)
* [SPARK-27514][SQL] Skip collapsing windows with empty window expressions (apache#538)
* Bump hadoop to 2.9.2-palantir.5 (apache#537)
turboFei added a commit to turboFei/spark that referenced this pull request Nov 6, 2025
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.

3 participants