Incorporate vitaly-t changes

[drew-gross]

@vitaly-t this is the replacement PR. Everything looks good to me. Thanks for the help! Tests already passed on the original PR so I will merge right away.

[vitaly-t]

@drew-gross

I don't know the role of this piece of code, but does it need performance and efficiency, or those are not important in this context?

I'm asking to see what other improvements might be worthwhile adding, because on the overall there could be too many ;)

[drew-gross]

Cool! This is definitely a performance-critical part of the system. getClass, updateObjectsByQuery, and deleteObjectsByQuery are probably the most critical. Of course, there are still many unimplemented parts, so until those are implemented there won't be much benefit in making stuff faster.

[vitaly-t]

Great! Let me know if you have any questions, I will be happy to help!

In the meantime, the biggest omission I've seen in the code is in trying to generate inserts and updates manually, not automatically with methods in the helpers namespace, which is exactly what it was implemented for ;)

That namespace lets you generate inserts and updates much faster, simpler and more reliably ;)

It is especially helpful in generating multi-row inserts and updates ;)

[vitaly-t]

P.S. Although, if it already works, then it's not that important, just for the future ;)

And just so, a shorter form for :name format is simply ~ ;) and ^ is short version for :raw ;)

[drew-gross]

That looks really useful, I will check out the helpers namespace. Awesome.

I prefer the long-form names, actually. Otherwise you can end up with strings of like 10 symbols in a row, which I think is harder to read.