Rewrite of QueryResultParser to fix issue #98. #100

Merged
merged 1 commit into from Dec 12, 2012

Conversation

Projects
None yet
2 participants
Contributor

pgelinas commented Dec 11, 2012

No description provided.

I remember that totalrows may not correspond to the number of rows returned by the query. it is the total number of rows in the view. so we should probably just init the array list with the default constructor.

helun added a commit that referenced this pull request Dec 12, 2012

Merge pull request #100 from pgelinas/fix-issue-98
Rewrite of QueryResultParser to fix issue #98.

@helun helun merged commit 5c54ba4 into helun:master Dec 12, 2012

1 check passed

default The Travis build passed
Details
Contributor

pgelinas commented Dec 12, 2012

Indeed, I didn't catch that, I'll update my branch with the change.

@pgelinas pgelinas referenced this pull request Dec 12, 2012

Merged

Updated PR #100 #103

helun added a commit that referenced this pull request Dec 12, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment