Replace getMetadata for row count with getQueryRowCount (which uses =COUNT(*))#65
Replace getMetadata for row count with getQueryRowCount (which uses =COUNT(*))#65dmpe wants to merge 1 commit intoChicago:devfrom
Conversation
f8b901c to
226eb7c
Compare
|
Hi @tomschenkjr, Yeeh, most probably total mess. But, on the other hand, I fixed the error message with the one about curl and that the host could not have been resolved. :) Let me know how can I help more in order to merge this PR. (I guess, you cannot merge it, right ?) |
|
So, I'd like to propose that we step back a bit and increment through these new features a bit more incrementally. We can add a feature and make a pull request before moving onto the next one--break-up the larger pull request into smaller ones. I've opened a Gitters forum, which will let us discuss this a little more easily. I am thinking we work on features in these order:
To get ready for this, I would recommend:
We should chat through this, though, to make sure it makes sense. If everyone is interested in this approach, jump into the forum and we can plan it out. We'll have to redo some of this work, but I think it's needed. CC'ing @marks and @geneorama |
|
@tomschenkjr @dmpe - this (and nearly anything you all would decide) is completely A-OK with me. As I said in the Gitter channel:
|
#64 (comment)
this now takes care of: