kmozaid commented on a change in pull request #8224:
URL: https://github.com/apache/pinot/pull/8224#discussion_r814430771
##########
File path:
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/partition/metadata/ColumnPartitionMetadata.java
##########
@@ -52,11 +57,14 @@
* @param functionName Name of the partition function
* @param numPartitions Number of total partitions for this column
* @param partitions Set of partitions the column contains
+ * @param functionConfig Configuration required by partition function.
*/
- public ColumnPartitionMetadata(String functionName, int numPartitions,
Set<Integer> partitions) {
+ public ColumnPartitionMetadata(String functionName, int numPartitions,
Set<Integer> partitions,
+ Map<String, String> functionConfig) {
Review comment:
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
