You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "namrathamyske (via GitHub)" <gi...@apache.org> on 2023/06/26 23:06:13 UTC
[GitHub] [iceberg] namrathamyske opened a new issue, #7918: Iceberg requiredNumOfPartitions method
namrathamyske opened a new issue, #7918:
URL: https://github.com/apache/iceberg/issues/7918
### Feature Request / Improvement
Hello community,
I wanted to override method in SparkWrite.java
` default int requiredNumPartitions() {
return 0;
}
`
The primary goal of this method is pass custom number of partitions to the range/hash partitioner (After taking value from spark configs) Anyone has concerns on this?
@rdblue @RussellSpitzer @aokolnychyi @jackye1995
### Query engine
None
--
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@iceberg.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
[GitHub] [iceberg] namrathamyske commented on issue #7918: Iceberg requiredNumOfPartitions method
Posted by "namrathamyske (via GitHub)" <gi...@apache.org>.
namrathamyske commented on issue #7918:
URL: https://github.com/apache/iceberg/issues/7918#issuecomment-1615208124
@rdblue @RussellSpitzer @aokolnychyi
Any specific reason we pass the finalNumPartitions from iceberg? spark is not able to apply AQE coalesce partitions as we are specifying the number of partitions(leading to small files problem) But when we pass None as the number of partitions, Spark AQE kicks in and it coalesces partitions as suitable - [https://github.com/apache/iceberg/blob/8af5b1d8c032ff2b0b2d738b3b5f791e16b83e84/sp[…]ution/datasources/v2/ExtendedDistributionAndOrderingUtils.scala](https://github.com/apache/iceberg/blob/8af5b1d8c032ff2b0b2d738b3b5f791e16b83e84/sp[%E2%80%A6]ution/datasources/v2/ExtendedDistributionAndOrderingUtils.scala).
--
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@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [I] Iceberg requiredNumOfPartitions method [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed issue #7918: Iceberg requiredNumOfPartitions method
URL: https://github.com/apache/iceberg/issues/7918
--
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@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [I] Iceberg requiredNumOfPartitions method [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #7918:
URL: https://github.com/apache/iceberg/issues/7918#issuecomment-2357237403
This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale'
--
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@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [I] Iceberg requiredNumOfPartitions method [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #7918:
URL: https://github.com/apache/iceberg/issues/7918#issuecomment-2325403806
This issue has been automatically marked as stale because it has been open for 180 days with no activity. It will be closed in next 14 days if no further activity occurs. To permanently prevent this issue from being considered stale, add the label 'not-stale', but commenting on the issue is preferred when possible.
--
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@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org