-
Notifications
You must be signed in to change notification settings - Fork 230
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
Show partition metrics for custom shuffler reader #1060
Show partition metrics for custom shuffler reader #1060
Conversation
sql-plugin/src/main/scala/org/apache/spark/sql/rapids/execution/ShuffledBatchRDD.scala
Outdated
Show resolved
Hide resolved
948d9be
to
4eaa240
Compare
sql-plugin/src/main/scala/org/apache/spark/sql/rapids/execution/ShuffledBatchRDD.scala
Outdated
Show resolved
Hide resolved
build |
build |
5a9fd23
to
69c39a7
Compare
build |
failed due to timeout this time |
build |
orc write test failed this time
|
Need to upmerge this PR. That test was recently fixed. |
Signed-off-by: Andy Grove <andygrove@nvidia.com>
Signed-off-by: Andy Grove <andygrove@nvidia.com>
Signed-off-by: Andy Grove <andygrove@nvidia.com>
69c39a7
to
ed7272c
Compare
build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Protip: If you merge to the latest on branch-0.3 rather than rebasing and force-pushing, Github is smart enough to not lose the PR approval. See #1145 as an example.
Now failing to build against 3.1.0 due to changes to MapOutputTracker API. I may need to do some shim work. I will look at this tomorrow. |
Signed-off-by: Andy Grove <andygrove@nvidia.com>
build |
build |
Failed due to |
build |
* Add partition metrics to custom shuffle reader Signed-off-by: Andy Grove <andygrove@nvidia.com> * revert change Signed-off-by: Andy Grove <andygrove@nvidia.com> * refactor to combine metrics and reader in single match statement Signed-off-by: Andy Grove <andygrove@nvidia.com> * use shim layer to get map output sizes Signed-off-by: Andy Grove <andygrove@nvidia.com>
* Add partition metrics to custom shuffle reader Signed-off-by: Andy Grove <andygrove@nvidia.com> * revert change Signed-off-by: Andy Grove <andygrove@nvidia.com> * refactor to combine metrics and reader in single match statement Signed-off-by: Andy Grove <andygrove@nvidia.com> * use shim layer to get map output sizes Signed-off-by: Andy Grove <andygrove@nvidia.com>
…IDIA#1060) Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>
This PR adds metrics to
GpuCustomShuffleReader
so that we can see the number and size of the resulting partitions.