Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changes to how cxtracker.pl stores sessions in MySQL #14

Open
gamelinux opened this issue Jan 13, 2013 · 4 comments
Open

Changes to how cxtracker.pl stores sessions in MySQL #14

gamelinux opened this issue Jan 13, 2013 · 4 comments
Assignees
Milestone

Comments

@gamelinux
Copy link
Owner

Change from using:

MERGE and MyISAM
to
VIEW and InnoDB

@ghost ghost assigned gamelinux Jan 13, 2013
@wmesser
Copy link
Contributor

wmesser commented Jan 14, 2013

See patchfile in Issue #13

@wmesser
Copy link
Contributor

wmesser commented Jan 15, 2013

DBD::mysql::st execute failed: Column 'sid' in field list is ambiguous at bin/cxtracker2db.pl line 684, <FILE> line 1.

I think we only get this when having multiple tables. It's got to be something I did wrong in the way the VIEW is created.

Putting it here for easier tracking for the time being.

@wmesser
Copy link
Contributor

wmesser commented Jan 15, 2013

This may be a stupid question, but where is that value of 'sid' even coming from? It's not in the stats files. We already have a session ID. Is sid just the hostname? If so, it is definitely not supposed to be numeric.

@wmesser
Copy link
Contributor

wmesser commented Jan 15, 2013

I changed it to VARCHAR(255), and it's at least logging the hostname correctly, but it's still 'ambiguous' and bug persists.

gamelinux pushed a commit that referenced this issue Feb 21, 2014
Cleaned up comments and added support for OpenFPC integration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants