[jira] [Commented] (ARROW-5311) [C++] Return more specific invalid Status in Take kernel

2019-05-15 Thread Antoine Pitrou (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-5311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16840454#comment-16840454 ] Antoine Pitrou commented on ARROW-5311: --- {{KeyError}} is more for column or field names etc. >

[jira] [Commented] (ARROW-5311) [C++] Return more specific invalid Status in Take kernel

2019-05-15 Thread Joris Van den Bossche (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-5311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16840410#comment-16840410 ] Joris Van den Bossche commented on ARROW-5311: -- With IndexError, we could match typical

[jira] [Commented] (ARROW-5311) [C++] Return more specific invalid Status in Take kernel

2019-05-15 Thread Benjamin Kietzman (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-5311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16840392#comment-16840392 ] Benjamin Kietzman commented on ARROW-5311: -- Alternatively, we could just use KeyError > [C++]

[jira] [Commented] (ARROW-5311) [C++] Return more specific invalid Status in Take kernel

2019-05-14 Thread Wes McKinney (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-5311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16839385#comment-16839385 ] Wes McKinney commented on ARROW-5311: - Seems OK to me, interested in other opinions > [C++] Return

[jira] [Commented] (ARROW-5311) [C++] Return more specific invalid Status in Take kernel

2019-05-14 Thread Antoine Pitrou (JIRA)
[ https://issues.apache.org/jira/browse/ARROW-5311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16839210#comment-16839210 ] Antoine Pitrou commented on ARROW-5311: --- I think we could probably add a Status::IndexError on the