Standardize collection names

[engelgabriel]

All to lowercase, and remove data. prefix

[rwakida]

Questions:

1. Does data.ChatRoom becomes chat_room or chatroom?
2. Any other collections besides: data.ChatMessage, data.ChatRoom, data.ChatSubscription, and data.ChatTyping?
3. Does the solution need to migrate existing data? Meteor doesn't support renaming or deleting a collection as far as I know. Any suggestions?

[sampaiodiego]

I was thinking in to use a prefic like "rocket" for collection names. should be one of these:

• rocket.chatRoom, rocket.chatMessage, rocket.chatSubscription, ...
• rocketRoom, rocketMessage, rocketSubscription, ...

I think using a prefix is the best for an existing database with other collections.

What do you others think?

[engelgabriel]

I think we should either lose the prefix all together.. and use

• rooms
• messages
• subscriptions

or

• rocketchat_rooms
• rocketchat_messages
• rocketchat_subscriptions

[rwakida]

As an outsider learning the system, the prefix helps to differentiate Rocket.Chat's collections from non-Rocket.Chat collections. I like rocketchat_rooms format.

What about existing data migration?

[rodrigok]

I agree with @rwakida.

About the data migration we already use one lib to do migrations, you can see here (https://github.com/RocketChat/Rocket.Chat/tree/master/server/startup/migrations)

[rwakida]

Thanks @rodrigok. I saw the migrations, but I think this is a special case. Meteor (from what I know) doesn't support Mongo's renameCollection or drop collection. So we either have to:

1. Instruct the user to manually run renameCollection from the shell before starting meteor.
   or
2. Rename the collections in collections.coffee. Create a migration that accesses the old collections and migrates the data to the new collections.

The first option is clean, but requires user intervention. The second option is user-friendly, but leaves the old collections and the user has to manually drop the old collections.

A Google search suggested using a Node package that provides shell access, but only works when NOT using the embedded MongoDB.

I'm relatively new to Meteor so please excuse me if I missed something.

[sampaiodiego]

@rodrigok but using meteor could be painful since you can't rename collections