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

String or Binary data will be truncated #4750

Merged
merged 1 commit into from
Sep 27, 2018

Conversation

tboggiano
Copy link
Contributor

Physical machine is 16 characters long rather than 15 characters. Changed definition of @sys_info table. Also, worth noting for future a Virtual Box VM in Linux shows up as a Physical Machine.

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

Physical machine is 16 characters long rather than 15 characters.  Changed definition of @sys_info table. Also, worth noting for future a Virtual Box VM in Linux shows up as a Physical Machine.
@danielnelson
Copy link
Contributor

@m82labs Can you review?

@danielnelson danielnelson added fix pr to fix corresponding bug area/sqlserver labels Sep 27, 2018
@m82labs
Copy link
Contributor

m82labs commented Sep 27, 2018 via email

@danielnelson danielnelson added this to the 1.8.1 milestone Sep 27, 2018
@danielnelson danielnelson merged commit 69f6612 into influxdata:master Sep 27, 2018
danielnelson pushed a commit that referenced this pull request Sep 27, 2018
rgitzel pushed a commit to rgitzel/telegraf that referenced this pull request Oct 17, 2018
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
dupondje pushed a commit to dupondje/telegraf that referenced this pull request Apr 22, 2019
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/sqlserver fix pr to fix corresponding bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants