{"action":"created","issue":{"url":"https://api.github.com/repos/influxdb/influxdb-scala/issues/4","labels_url":"https://api.github.com/repos/influxdb/influxdb-scala/issues/4/labels{/name}","comments_url":"https://api.github.com/repos/influxdb/influxdb-scala/issues/4/comments","events_url":"https://api.github.com/repos/influxdb/influxdb-scala/issues/4/events","html_url":"https://github.com/influxdb/influxdb-scala/issues/4","id":47680687,"number":4,"title":"API Cleanup: A couple ideas","user":{"login":"velvia","id":1062875,"avatar_url":"https://avatars.githubusercontent.com/u/1062875?v=3","gravatar_id":"","url":"https://api.github.com/users/velvia","html_url":"https://github.com/velvia","followers_url":"https://api.github.com/users/velvia/followers","following_url":"https://api.github.com/users/velvia/following{/other_user}","gists_url":"https://api.github.com/users/velvia/gists{/gist_id}","starred_url":"https://api.github.com/users/velvia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/velvia/subscriptions","organizations_url":"https://api.github.com/users/velvia/orgs","repos_url":"https://api.github.com/users/velvia/repos","events_url":"https://api.github.com/users/velvia/events{/privacy}","received_events_url":"https://api.github.com/users/velvia/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2014-11-04T06:01:28Z","updated_at":"2015-03-05T10:59:52Z","closed_at":null,"body":"Hey guys,\r\n\r\nI'm liking InfluxDB so far but the API for this makes me slightly sad.\r\n\r\n- Can we have everything accept `Seq[...]` instead of `Array[...]`?   Seqs are much more flexible and easier to work with.\r\n- It's not considered good Scala style to have var's especially in the Client class constructor.   You don't really want to allow someone to change the database name in the middle of using the client.\r\n- I think a much better API than `Array[Array[Any]]` would be to have WriteSeries take case classes.   Scala macros can deduce the fields from the case classes, and this way it is type safe.  Using a `Array[Any]` forces boxing of all numeric values, which is very slow and expensive, and it is not type safe at all.\r\n\r\nI might be interested in submitting PRs but wanted to run these things by you guys first.\r\n\r\nAlso, how about packaging and submitting jars to Maven Central, or at least Bintray (much easier than Maven Central)?"},"comment":{"url":"https://api.github.com/repos/influxdb/influxdb-scala/issues/comments/77344975","html_url":"https://github.com/influxdb/influxdb-scala/issues/4#issuecomment-77344975","issue_url":"https://api.github.com/repos/influxdb/influxdb-scala/issues/4","id":77344975,"user":{"login":"mfirry","id":1107071,"avatar_url":"https://avatars.githubusercontent.com/u/1107071?v=3","gravatar_id":"","url":"https://api.github.com/users/mfirry","html_url":"https://github.com/mfirry","followers_url":"https://api.github.com/users/mfirry/followers","following_url":"https://api.github.com/users/mfirry/following{/other_user}","gists_url":"https://api.github.com/users/mfirry/gists{/gist_id}","starred_url":"https://api.github.com/users/mfirry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mfirry/subscriptions","organizations_url":"https://api.github.com/users/mfirry/orgs","repos_url":"https://api.github.com/users/mfirry/repos","events_url":"https://api.github.com/users/mfirry/events{/privacy}","received_events_url":"https://api.github.com/users/mfirry/received_events","type":"User","site_admin":false},"created_at":"2015-03-05T10:59:52Z","updated_at":"2015-03-05T10:59:52Z","body":"You're right. \r\nI still do believe `Seqs` are more \"everyday toys\" for Scala developers rather than Arrays. Plus scala.Predef provides handy implicit conversions. \r\n\r\nTalking about PR, there was in fact a PR a while ago by @fsauer65  but was let go because it was using Scala 2.11.\r\n\r\nI'll see if I can send you a PR before the end of this week. \r\n\r\n@velvia any chance we can work on this together?\r\n\r\n"}}