Support rates >17.1gb/s #3754

Merged
merged 2 commits into from Jun 30, 2016

Projects

None yet

4 participants

@pblasquez
Contributor

No description provided.

@pblasquez pblasquez Support rates >17.1gb/s
2759bf0
@f0o
Member
f0o commented Jun 30, 2016

Oh boy, this will kill my install again haha.

Just writing a comment so I will be informed of the merge

@laf
Member
laf commented Jun 30, 2016

I chose to run this as one query which I think this should be updated to do:

mysql> ALTER TABLE `ports` CHANGE `ifInOctets_rate` `ifInOctets_rate` BIGINT(20) NULL DEFAULT NULL, CHANGE `ifOutOctets_rate` `ifOutOctets_rate` BIGINT(20) NULL DEFAULT NULL ;
Query OK, 69406 rows affected (13.43 sec)
Records: 69406  Duplicates: 0  Warnings: 0

+1 from me.

@laf laf added the Schema label Jun 30, 2016
@f0o
Member
f0o commented Jun 30, 2016

Yeah one statement instead of two is better

@laf
Member
laf commented Jun 30, 2016

@f0o you may not have an issue with this one, the last update took over a minute for me where as this was pretty quick

@pblasquez pblasquez Changed to one line
61544e5
@laf laf merged commit bd7b4fe into librenms:master Jun 30, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@f0o
Member
f0o commented Jul 1, 2016

We'll see, at least I'm aware of this schema update so I'll be on alert ;)

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