Skip to content

[SPARK-15291][GraphX] Remove redundant operations in SVD++ #13075

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[SPARK-15291][GraphX] Remove redundant operations in SVD++ #13075

wants to merge 1 commit into from

Conversation

zhengruifeng
Copy link
Contributor

@zhengruifeng zhengruifeng commented May 12, 2016 •
edited
Loading

What changes were proposed in this pull request?

val newVertices = g.vertices.mapValues(v => (v._1.toArray, v._2.toArray, v._3, v._4))
(Graph(newVertices, g.edges), u)

g is of type Graph[(Array[Double], Array[Double], Double, Double), Double],
and the return value seems just equal to (g, u)

How was this patch tested?

unit test

@SparkQA
Copy link

SparkQA commented May 12, 2016

Test build #58472 has finished for PR 13075 at commit 7e62b55.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@zhengruifeng zhengruifeng changed the title [TEST] Remove redundant codes in SVD++ [SPARK-15291][GraphX] Remove redundant codes in SVD++ May 12, 2016
@zhengruifeng zhengruifeng changed the title [SPARK-15291][GraphX] Remove redundant codes in SVD++ [SPARK-15291][GraphX] Remove redundant operations in SVD++ May 12, 2016
@zhengruifeng zhengruifeng deleted the svdpp_refine branch May 27, 2016 23:53
@panbingkun panbingkun mentioned this pull request May 26, 2024
Closed
LuciferYang pushed a commit that referenced this pull request May 28, 2024
@panbingkun @LuciferYang
[SPARK-48420][BUILD] Upgrade netty to 4.1.110.Final
a88cc1a 
### What changes were proposed in this pull request?
The pr aims to upgrade `netty` from `4.1.109.Final` to `4.1.110.Final`.

### Why are the changes needed?
- https://netty.io/news/2024/05/22/4-1-110-Final.html
  This version has brought some bug fixes and improvements, such as:
  Fix Zstd throws Exception on read-only volumes (netty/netty#13982)
  Add unix domain socket transport in netty 4.x via JDK16+ ([#13965](netty/netty#13965))
  Backport #13075: Add the AdaptivePoolingAllocator ([#13976](netty/netty#13976))
  Add no-value key handling only for form body ([#13998](netty/netty#13998))
  Add support for specifying SecureRandom in SSLContext initialization ([#14058](netty/netty#14058))

- https://github.com/netty/netty/issues?q=milestone%3A4.1.110.Final+is%3Aclosed

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Pass GA.

### Was this patch authored or co-authored using generative AI tooling?
No.

Closes #46744 from panbingkun/SPARK-48420.

Authored-by: panbingkun <panbingkun@baidu.com>
Signed-off-by: yangjie01 <yangjie01@baidu.com>
riyaverm-db pushed a commit to riyaverm-db/spark that referenced this pull request Jun 7, 2024
@panbingkun @riyaverm-db
[SPARK-48420][BUILD] Upgrade netty to 4.1.110.Final
6def306 
### What changes were proposed in this pull request?
The pr aims to upgrade `netty` from `4.1.109.Final` to `4.1.110.Final`.

### Why are the changes needed?
- https://netty.io/news/2024/05/22/4-1-110-Final.html
  This version has brought some bug fixes and improvements, such as:
  Fix Zstd throws Exception on read-only volumes (netty/netty#13982)
  Add unix domain socket transport in netty 4.x via JDK16+ ([apache#13965](netty/netty#13965))
  Backport apache#13075: Add the AdaptivePoolingAllocator ([apache#13976](netty/netty#13976))
  Add no-value key handling only for form body ([apache#13998](netty/netty#13998))
  Add support for specifying SecureRandom in SSLContext initialization ([apache#14058](netty/netty#14058))

- https://github.com/netty/netty/issues?q=milestone%3A4.1.110.Final+is%3Aclosed

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Pass GA.

### Was this patch authored or co-authored using generative AI tooling?
No.

Closes apache#46744 from panbingkun/SPARK-48420.

Authored-by: panbingkun <panbingkun@baidu.com>
Signed-off-by: yangjie01 <yangjie01@baidu.com>
szehon-ho pushed a commit to szehon-ho/spark that referenced this pull request Aug 7, 2024
@panbingkun @dongjoon-hyun
[SPARK-48420][BUILD] Upgrade netty to 4.1.110.Final
8278aa7 
The pr aims to upgrade `netty` from `4.1.109.Final` to `4.1.110.Final`.

- https://netty.io/news/2024/05/22/4-1-110-Final.html
  This version has brought some bug fixes and improvements, such as:
  Fix Zstd throws Exception on read-only volumes (netty/netty#13982)
  Add unix domain socket transport in netty 4.x via JDK16+ ([apache#13965](netty/netty#13965))
  Backport apache#13075: Add the AdaptivePoolingAllocator ([apache#13976](netty/netty#13976))
  Add no-value key handling only for form body ([apache#13998](netty/netty#13998))
  Add support for specifying SecureRandom in SSLContext initialization ([apache#14058](netty/netty#14058))

- https://github.com/netty/netty/issues?q=milestone%3A4.1.110.Final+is%3Aclosed

No.

Pass GA.

No.

Closes apache#46744 from panbingkun/SPARK-48420.

Authored-by: panbingkun <panbingkun@baidu.com>
Signed-off-by: yangjie01 <yangjie01@baidu.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zhengruifeng @SparkQA