can't update with coll = {database(), collection()}

[lubberland]

version v0.8.3 , mongo version 3.2.1
test code:

{ok, Conn} = mc_worker_api:connect ([{database, <<"admin">>}, {host, "127.0.0.1"}, {port, 9999}])
Collect = {<<"test">>, <<"device">>},
Command = #{<<"123">> => pid_to_list(self())},
mc_worker_api:update(Conn, Collect, {<<"abc">>, <<"abc">>}, #{<<"$set">> => bson:flatten_map(Command)}).

return msg:

{false,#{<<"code">> => 9,
         <<"errmsg">> => <<"wrong type for 'update' field, expected string, found update: { test: \"device\" }">>}}

do i use the wrong way? thank you

[comtihon]

Why you are using collection {<<"test">>, <<"device">>}. What do you want to do with this tuple?
Collection supposed to be binary.

[lubberland]

i have to connect mongodb with account and password, so i have to use "admin" database at the beginning
mongo cmd:
./mongo --port 27017 -u myUserAdmin -p abc123 --authenticationDatabase admin
code:

mc_worker_api:connect ([{database, <<"admin">>}, {host, "127.0.0.1"}, {port, 9999}, {login, <<"myUserAdmin ">>}, {password, <<"abc123">> }])

but i want to update insert delete data with another database, so i use collection {<<"test">>, <<"device">>}
Do you have any good suggestions? thank you

[alttagil]

I don't know when and if this feature will be implemented here. It actually was. But while you wait for an answer here, you can check https://github.com/alttagil/mongodb-erlang repository where this feature is implemented right now.

[comtihon]

Yes, is it not supported in this driver due to changing api commands.

[henryj]

Are we gonna have this support back? it's really critical for us as well.

[henryj]

btw, the api commands change also breaks insert/update/delete/etc support for mongo 2.4 (probably < 3.0) (insert/update/delete $cmd are added in 2.6). Maybe passing all those operations to "command" in the api is not such a good idea? as alttagil's fork doesn't and both colldb() and 2.4 support are working.

@alttagil in your fork, you should expose command/4 in mongoc so commands to other dbs can work. :)

[henryj]

@comtihon would you please review the pr and merge. Thanks.

[alttagil]

@henryj ok )