Apply suggestions from code review

Co-authored-by: Mark Harris <[email protected]>

cpp/include/cudf/detail/aggregation/aggregation.hpp

@@ -1497,8 +1497,7 @@ AGG_KIND_MAPPING(aggregation::VARIANCE, var_aggregation);
  *
  * @tparam F Type of callable
  * @param k The `aggregation::Kind` value to dispatch
- * @param f The callable that accepts an `aggregation::Kind` non-type template
- * argument.
+ * @param f The callable that accepts an `aggregation::Kind` callable function object.
  * @param args Parameter pack forwarded to the `operator()` invocation
  * @return Forwards the return value of the callable.
  */
@@ -1626,8 +1625,7 @@ struct dispatch_source {
  * parameter of the callable `F`
  * @param k The `aggregation::Kind` used to dispatch an `aggregation::Kind`
  * non-type template parameter for the second template parameter of the callable
- * @param f The callable that accepts `data_type` and `aggregation::Kind` non-type template
- * arguments.
+ * @param f The callable that accepts `data_type` and `aggregation::Kind` function object.
  * @param args Parameter pack forwarded to the `operator()` invocation
  * `F`.
  */
@@ -1647,7 +1645,7 @@ CUDF_HOST_DEVICE inline constexpr decltype(auto) dispatch_type_and_aggregation(d
  * performed on elements of type  source_type.
  *
  * @param source_type The element type to be aggregated
- * @param k The aggregation
+ * @param k The aggregation kind
  * @return data_type The target_type of  k performed on  source_type
  * elements
  */