Log missing keys on debug level

[bastianjoel]

resolves #269

[ostcar]

Its ok for me. But I think, this will create strange messages, when there are more then one invalid keys in the message bus.

I would propose, that you change createKeyList that it does not return an error anymore, but as a second argument a list of invalid keys.

func createKeyList(collection string, id int, fields []string) ([]dskey.Key, []dskey.Key)

Then you can create a better debug-error message with it.

It also makes it more clear, that you don't ignore a generic error, but only invalid keys.

[ostcar]

Its ok for me. But I think, this will create strange messages, when there are more then one invalid keys in the message bus.

I would propose, that you change createKeyList that it does not return an error anymore, but as a second argument a list of invalid keys.

func createKeyList(collection string, id int, fields []string) ([]dskey.Key, []dskey.Key)

Then you can create a better debug-error message with it.

It also makes it more clear, that you don't ignore a generic error, but only invalid keys.

Of cause, it can not return invalid keys, but only the invalid fields:
func createKeyList(collection string, id int, fields []string) ([]dskey.Key, []string)

[bastianjoel]

I would propose, that you change createKeyList that it does not return an error anymore, but as a second argument a list of invalid keys.

I prefer not to do that right now as IMHO this would also mean that we should change FromParts return value to (Key, bool). This however requires changes in other services. Right now it returns an error and therefore we should pass that error in case of any future changes.