You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@uniffle.apache.org by GitBox <gi...@apache.org> on 2022/11/12 07:47:52 UTC

[GitHub] [incubator-uniffle] zuston opened a new issue, #316: [Improvement] Remove the PartitionRange

zuston opened a new issue, #316:
URL: https://github.com/apache/incubator-uniffle/issues/316

   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   
   
   ### Search before asking
   
   - [X] I have searched in the [issues](https://github.com/apache/incubator-uniffle/issues?q=is%3Aissue) and found no similar issues.
   
   
   ### What would you like to be improved?
   
   Currently, the partitionRange is not used in spark/mr. Let's remove this to reduce the complexity
   
   ### How should we improve?
   
   Delete all related code.
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@uniffle.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-uniffle] zuston commented on issue #316: [Improvement] Remove the PartitionRange

Posted by GitBox <gi...@apache.org>.
zuston commented on issue #316:
URL: https://github.com/apache/incubator-uniffle/issues/316#issuecomment-1312402324

   This is mentioned in offline discussion @jerqi . 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@uniffle.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-uniffle] zuston commented on issue #316: [Improvement] Remove the PartitionRange

Posted by "zuston (via GitHub)" <gi...@apache.org>.
zuston commented on issue #316:
URL: https://github.com/apache/incubator-uniffle/issues/316#issuecomment-1413080240

   Closed. Feel free to open if having any problem.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@uniffle.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@uniffle.apache.org
For additional commands, e-mail: issues-help@uniffle.apache.org


[GitHub] [incubator-uniffle] zuston closed issue #316: [Improvement] Remove the PartitionRange

Posted by "zuston (via GitHub)" <gi...@apache.org>.
zuston closed issue #316: [Improvement] Remove the PartitionRange 
URL: https://github.com/apache/incubator-uniffle/issues/316


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@uniffle.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@uniffle.apache.org
For additional commands, e-mail: issues-help@uniffle.apache.org


[GitHub] [incubator-uniffle] advancedxy commented on issue #316: [Improvement] Remove the PartitionRange

Posted by GitBox <gi...@apache.org>.
advancedxy commented on issue #316:
URL: https://github.com/apache/incubator-uniffle/issues/316#issuecomment-1346320250

   Let's not pursue this for now? I may have ideas on how to reduce small I/Os when involved in many small tasks scenario, it may require partition range to merge small I/Os.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@uniffle.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-uniffle] zuston commented on issue #316: [Improvement] Remove the PartitionRange

Posted by GitBox <gi...@apache.org>.
zuston commented on issue #316:
URL: https://github.com/apache/incubator-uniffle/issues/316#issuecomment-1346322687

   > Let's not pursue this for now? I may have ideas on how to reduce small I/Os when involved in many small tasks scenario, it may require partition range to merge small I/Os.
   
   OK.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@uniffle.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org