Skip to content

SQL Server - on-prem queries refactor (again)#8323

Merged
ssoroka merged 1 commit intoinfluxdata:masterfrom
Trovalo:onprem-refactor-again
Oct 27, 2020
Merged

SQL Server - on-prem queries refactor (again)#8323
ssoroka merged 1 commit intoinfluxdata:masterfrom
Trovalo:onprem-refactor-again

Conversation

@Trovalo
Copy link
Copy Markdown
Contributor

@Trovalo Trovalo commented Oct 27, 2020

Required for all PRs:

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

Sorry to bother you (again) with this @ssoroka but I've noticed this just now.
looks like that the edits made with #8172 to the main file of the sqlserver on prem queries have been lost in the merge... somehow.
If I go to that or (#8172) I can see that only 2 files have been edited, and "plugins/inputs/sqlserver/sqlserverqueries.go" is not included.

Therefore I made this PR to re-add all the missing formatting and edits that had been made to that file

@ssoroka
Copy link
Copy Markdown
Contributor

ssoroka commented Oct 27, 2020

probably a bad conflict resolution somewhere. Do you want this in 1.16.1?

@ssoroka ssoroka merged commit 0621b4b into influxdata:master Oct 27, 2020
@Trovalo
Copy link
Copy Markdown
Contributor Author

Trovalo commented Oct 27, 2020

probably a bad conflict resolution somewhere. Do you want this in 1.16.1?

Yes

@Trovalo Trovalo deleted the onprem-refactor-again branch October 27, 2020 15:51
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
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

Successfully merging this pull request may close these issues.

2 participants