Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/5731#issuecomment-126877346
@jkbradley There is already an IntArrayParam in the sharedParam.
Besides, there are some issues to talk:
- Should we consider the scenario that some attributes have both indices
and names, while others have only one of them? Say, a vector `(1, 2, 3, 4, 5)`,
while attribtues are `((0, "first"), (1, "second"), (2), ("third"), (4))`, or
we assume beforehand that all attributes are completed, that is, including
names and indices.
- Should we retain the order that the end-user specified in the interface?
Say, `selectedIndices(3,2,5,1)` or `selectedNames("second", "first")`.
Mixed the vector with attributes, I think these two issues affect the
design of the code.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
