Skip to content
This repository has been archived by the owner on Nov 14, 2022. It is now read-only.

Change the variable name so that the passed parameters are correct #12

Merged
merged 4 commits into from
Feb 5, 2021
Merged

Change the variable name so that the passed parameters are correct #12

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
79cbf31
Create BlockManager.scala
JustDoCoder Jan 29, 2021
46cdfe0
Merge pull request #1 from oap-project/master
JustDoCoder Feb 2, 2021
2c213ff
Update BlockManager.scala
JustDoCoder Feb 2, 2021
21fae64
Update BlockManager.scala
JustDoCoder Feb 3, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/main/scala/org/apache/spark/storage/BlockManager.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -210,8 +210,8 @@ private[spark] class BlockManager(
}

require(file.isDirectory(), "PMem directory is required for initialization")
PersistentMemoryPlatform.initialize(initPath, pmemInitialSize, 0)
logInfo(s"Intel Optane PMem initialized with path: ${initPath}, size: ${pmemInitialSize} ")
PersistentMemoryPlatform.initialize(path_postfix, pmemInitialSize, 0)
Copy link

@winningsix winningsix Feb 2, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

file.getAbsolutePath ?

Copy link
Contributor Author

@JustDoCoder JustDoCoder Feb 3, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

@winningsix Thanks for reminding, the absolute path of the object should be passed as a parameter

logInfo(s"Intel Optane PMem initialized with path: ${path_postfix}, size: ${pmemInitialSize} ")
}
} else if (pmemMode.equals("KMemDax")) {
if (!isDriver) {
Expand Down