src/Microsoft.SqlTools.ServiceLayer/TableDesigner/TableDesignerService.cs (6 lines):
	- line 203: // TODO: handle add item to foreign key's column mapping
	- line 213: // TODO: handle add item to index's column specification
	- line 264: // TODO: handle remove item from foreign key's column mapping
	- line 274: // TODO: handle remove item from index's column specification
	- line 434: // TODO: handle foreign key collection property update
	- line 589: // TODO: set other properties of the table


src/Microsoft.SqlTools.ServiceLayer/LanguageServices/LanguageService.cs (5 lines):
	- line 678: // TODO: need mechanism return errors from event handlers
	- line 704: // TODO: need mechanism return errors from event handlers
	- line 732: // TODO: need mechanism return errors from event handlers
	- line 859: // TODO: need mechanism return errors from event handlers
	- line 905: // TODO: need mechanism return errors from event handlers


src/Microsoft.SqlTools.ServiceLayer/Workspace/WorkspaceService.cs (5 lines):
	- line 42: /// TODO: Figure out how to make this truely singleton even with dependency injection for tests
	- line 243: // TODO: this probably means the ScriptFile model is in a bad state or out of sync with the actual file; we should recover here
	- line 277: // TODO: this probably means the ScriptFile model is in a bad state or out of sync with the actual file; we should recover here
	- line 313: // TODO: this probably means the ScriptFile model is in a bad state or out of sync with the actual file; we should recover here
	- line 338: // TODO: this probably means the ScriptFile model is in a bad state or out of sync with the actual file; we should recover here


src/Microsoft.Kusto.ServiceLayer/Workspace/WorkspaceService.cs (5 lines):
	- line 41: /// TODO: Figure out how to make this truely singleton even with dependency injection for tests
	- line 242: // TODO: this probably means the ScriptFile model is in a bad state or out of sync with the actual file; we should recover here
	- line 276: // TODO: this probably means the ScriptFile model is in a bad state or out of sync with the actual file; we should recover here
	- line 312: // TODO: this probably means the ScriptFile model is in a bad state or out of sync with the actual file; we should recover here
	- line 337: // TODO: this probably means the ScriptFile model is in a bad state or out of sync with the actual file; we should recover here


src/Microsoft.SqlTools.ServiceLayer/Scripting/ScriptAsScriptingOperation.cs (4 lines):
	- line 79: // TODO: try to use one of the existing connections
	- line 93: // TODO: Not including the header by default. We have to get this option from client
	- line 524: // TODO: get the default schema
	- line 534: // TODO: have to add Scripting behavior to Smo ScriptingOptions class


src/Microsoft.SqlTools.Hosting/Hosting/Protocol/MessageDispatcher.cs (4 lines):
	- line 279: //     // TODO: Message not supported error
	- line 298: // TODO: Message not supported error
	- line 303: //     // TODO: Return message not supported
	- line 336: // TODO: Dispose of anything?


src/Microsoft.SqlTools.ServiceLayer/DisasterRecovery/RestoreOperation/RestoreUtil.cs (4 lines):
	- line 291: //TODO: the code is moved from ssms and used for other typs of restore operation
	- line 553: //TODO: the code is moved from ssms and used for other typs of restore operation
	- line 593: //TODO: the code is moved from ssms and used for other typs of restore operation
	- line 727: value = "RowData";//TODO SR.RowData;


src/Microsoft.SqlTools.ServiceLayer/ShowPlan/ShowPlanGraph/ObjectWrapperTypeConverter.cs (3 lines):
	- line 125: // TODO: may need to generate a random property name
	- line 931: // TODO: review this - may need better condition
	- line 955: // TODO: review this - may need better condition


src/Microsoft.SqlTools.ServiceLayer/Utility/SqlScriptFormatters/ToSqlScript.cs (3 lines):
	- line 82: // TODO: This doesn't support UDTs at all.
	- line 83: // TODO: It's unclear if this will work on a case-sensitive db collation
	- line 201: // @TODO: to constants file


src/Microsoft.SqlTools.ManagedBatchParser/ReliableConnection/ReliableConnectionHelper.cs (3 lines):
	- line 166: TODO - re-enable if we port ConnectionStringSecurer
	- line 975: /* TODO - CloneAndOpenConnection() requires IClonable, which doesn't exist in .NET Core
	- line 1311: /* TODO - this relies on porting SqlAuthenticationMethodUtils


src/Microsoft.Kusto.ServiceLayer/LanguageServices/LanguageService.cs (3 lines):
	- line 503: // TODO: need mechanism return errors from event handlers
	- line 529: // TODO: need mechanism return errors from event handlers
	- line 650: // TODO: need mechanism return errors from event handlers


src/Microsoft.Kusto.ServiceLayer/ObjectExplorer/Nodes/TreeNode.cs (3 lines):
	- line 242: // TODO consider why solution explorer has separate Children and Items options
	- line 266: // TODO consider why solution explorer has separate Children and Items options
	- line 484: // TODO consider use of deferred children and if it's necessary


src/Microsoft.SqlTools.Hosting/Hosting/Protocol/Serializers/JsonRpcMessageSerializer.cs (3 lines):
	- line 55: // TODO: Check for jsonrpc version
	- line 78: // TODO: Throw parse error
	- line 92: // TODO: Throw parse error


src/Microsoft.SqlTools.ServiceLayer/DisasterRecovery/CommonUtilities.cs (3 lines):
	- line 433: // TODO: This is implemented as internal property in SMO.
	- line 526: /* TODO: This is needed for Restore
	- line 569: /* TODO: This is needed for Restore


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/Nodes/TreeNode.cs (3 lines):
	- line 242: // TODO consider why solution explorer has separate Children and Items options
	- line 266: // TODO consider why solution explorer has separate Children and Items options
	- line 387: // TODO consider use of deferred children and if it's necessary


src/Microsoft.SqlTools.ServiceLayer/EditData/EditSession.cs (3 lines):
	- line 395: // @TODO: Reinstate this code once we have an interface around file generation
	- line 512: // @TODO: Add support for transactional commits
	- line 537: // @TODO: Prevent edit sessions from being modified while a commit is in progress


src/Microsoft.Kusto.ServiceLayer/Scripting/ScriptAsScriptingOperation.cs (3 lines):
	- line 52: // TODO: Not including the header by default. We have to get this option from client
	- line 165: // TODO: get the default schema
	- line 175: // TODO: have to add Scripting behavior to Smo ScriptingOptions class


src/Microsoft.Kusto.ServiceLayer/Utility/SqlScriptFormatters/ToSqlScript.cs (3 lines):
	- line 82: // TODO: This doesn't support UDTs at all.
	- line 83: // TODO: It's unclear if this will work on a case-sensitive db collation
	- line 201: // @TODO: to constants file


src/Microsoft.Kusto.ServiceLayer/Localization/transXliff/sr.it.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="translated">--TODO: impostare qui i valori dei parametri.</target>


src/Microsoft.SqlTools.ServiceLayer/Localization/transXliff/sr.fr.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="new">-- TODO: Set parameter values here.</target>


src/Microsoft.SqlTools.ServiceLayer/Localization/sr.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="new">-- TODO: Set parameter values here.</target>


src/Microsoft.Kusto.ServiceLayer/Localization/sr.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="new">-- TODO: Set parameter values here.</target>


src/Microsoft.SqlTools.ServiceLayer/Localization/transXliff/sr.ja.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="translated">-- TODO: ここにパラメーター値を設定します。</target>


src/Microsoft.SqlTools.ServiceLayer/Localization/transXliff/sr.zh-hant.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="translated">-- TODO: 在此處設定參數值。</target>


src/Microsoft.Kusto.ServiceLayer/Localization/transXliff/sr.pt-br.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="translated">--TODO: defina os valores de parâmetro aqui.</target>


src/Microsoft.SqlTools.ManagedBatchParser/ReliableConnection/RetryPolicy.cs (2 lines):
	- line 62: //TODO Defect 1078447 Validate whether CommandTimeout needs to be used differently in schema/data scenarios
	- line 282: /* TODO - Error code does not exist in SqlException for .NET Core


src/Microsoft.Kusto.ServiceLayer/Localization/transXliff/sr.fr.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="new">-- TODO: Set parameter values here.</target>


src/Microsoft.SqlTools.ServiceLayer/ShowPlan/ShowPlanGraph/PropertyFactory.cs (2 lines):
	- line 694: // TODO: review this debug code
	- line 727: // TODO: review this debug code


src/Microsoft.Kusto.ServiceLayer/Localization/transXliff/sr.zh-hans.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="new">-- TODO: Set parameter values here.</target>


src/Microsoft.Kusto.ServiceLayer/Localization/transXliff/sr.es.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="translated">-- TODO: Establezca los valores de los parámetros aquí.</target>


src/Microsoft.Kusto.ServiceLayer/Localization/transXliff/sr.de.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="translated">--TODO: Hier Parameterwerte festlegen.</target>


src/Microsoft.SqlTools.ServiceLayer/Localization/transXliff/sr.ru.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="new">-- TODO: Set parameter values here.</target>


src/Microsoft.Kusto.ServiceLayer/Localization/transXliff/sr.ko.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="translated">-- TODO: 여기에서 매개 변수 값을 설정합니다.</target>


src/Microsoft.SqlTools.ServiceLayer/Localization/transXliff/sr.de.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="translated">--TODO: Hier Parameterwerte festlegen.</target>


src/Microsoft.Kusto.ServiceLayer/Localization/transXliff/sr.ja.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="translated">-- TODO: ここにパラメーター値を設定します。</target>


src/Microsoft.SqlTools.ServiceLayer/Localization/transXliff/sr.ko.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="translated">-- TODO: 여기에서 매개 변수 값을 설정합니다.</target>


src/Microsoft.SqlTools.ServiceLayer/Localization/transXliff/sr.zh-hans.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="new">-- TODO: Set parameter values here.</target>


src/Microsoft.Kusto.ServiceLayer/ObjectExplorer/DataSourceModel/ServerNode.cs (2 lines):
	- line 54: // TODO Domain and username is not yet supported on .Net Core.
	- line 61: // TODO Consider adding IsAuthenticatingDatabaseMaster check in the code and


src/Microsoft.SqlTools.ServiceLayer/Localization/transXliff/sr.it.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="translated">--TODO: impostare qui i valori dei parametri.</target>


src/Microsoft.Kusto.ServiceLayer/Localization/transXliff/sr.zh-hant.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="translated">-- TODO: 在此處設定參數值。</target>


src/Microsoft.SqlTools.ServiceLayer/Localization/transXliff/sr.pt-br.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="translated">--TODO: defina os valores de parâmetro aqui.</target>


src/Microsoft.Kusto.ServiceLayer/Localization/transXliff/sr.ru.xlf (2 lines):
	- line 1617: <source>-- TODO: Set parameter values here.</source>
	- line 1618: <target state="new">-- TODO: Set parameter values here.</target>


src/Microsoft.SqlTools.ResourceProvider.DefaultImpl/AzureAuthenticationManager.cs (2 lines):
	- line 179: // TODO is there more needed here?
	- line 312: // TODO can port this over from the VS implementation if needed, but for now disabling as we don't serialize / deserialize subscriptions


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/SmoModel/ServerNode.cs (2 lines):
	- line 80: // TODO Domain and username is not yet supported on .Net Core.
	- line 87: // TODO Consider adding IsAuthenticatingDatabaseMaster check in the code and


src/Microsoft.SqlTools.ServiceLayer/Localization/transXliff/sr.es.xlf (2 lines):
	- line 1622: <source>-- TODO: Set parameter values here.</source>
	- line 1623: <target state="translated">-- TODO: Establezca los valores de los parámetros aquí.</target>


src/Microsoft.SqlTools.ServiceLayer/DisasterRecovery/RestoreOperation/RestoreDatabaseTaskDataObject.cs (2 lines):
	- line 527: //TODO: get the value from DatabaseRestorePlanner.IsTailLogBackupPossible
	- line 574: //TODO: get the value from DatabaseRestorePlanner.IsTailLogBackupWithNoRecoveryPossible


src/Microsoft.SqlTools.ServiceLayer/Formatter/Impl/FormatContext.cs (1 line):
	- line 42: // TODO is TOKEN_PROC_SEMI a keyword?


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/Nodes/NodeTypes.cs (1 line):
	- line 11: // TODO Consider replacing this with an auto-gen'd version


src/Microsoft.SqlTools.ServiceLayer/Management/Common/ManagementActionBase.cs (1 line):
	- line 623: // TODO: uncomment next line when SMO server will have support for Cancel


src/Microsoft.SqlTools.Hosting/Hosting/Protocol/Serializers/V8MessageSerializer.cs (1 line):
	- line 94: //     // TODO: Parse error


src/Microsoft.SqlTools.ServiceLayer/QueryExecution/Contracts/SaveResultsRequest.cs (1 line):
	- line 123: //TODO: define config for save as JSON


src/Microsoft.Kusto.ServiceLayer/QueryExecution/Contracts/SelectionData.cs (1 line):
	- line 12: /// TODO: Remove this in favor of buffer range end-to-end


src/Microsoft.SqlTools.ResourceProvider.Core/Discovery/AzureDatabaseDiscoveryProvider.cs (1 line):
	- line 105: //        result = new ServiceResponse<DatabaseInstanceInfo>(); //TODO: add error that we couldn't find any default database provider


src/Microsoft.SqlTools.Hosting/Hosting/ServiceHostEditorOperations.cs (1 line):
	- line 61: // TODO: Set the last param back to true!


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/Nodes/ChildFactory.cs (1 line):
	- line 69: // TODO Consider whether Remove operations need to be supported


src/Microsoft.SqlTools.ResourceProvider.Core/Extensibility/ExtensionUtils.cs (1 line):
	- line 151: // TODO should we handle this differently?


src/Microsoft.SqlTools.ServiceLayer/EditData/SmoEditMetadataFactory.cs (1 line):
	- line 129: // TODO: Remove IsSupported check once SMO fixes broken IsMemoryOptimized scenario (TFS #10871823)


src/Microsoft.SqlTools.ResourceProvider.DefaultImpl/AzureUserAccount.cs (1 line):
	- line 54: // TODO probably should check the AllTenants field


src/Microsoft.SqlTools.ServiceLayer/EditData/UpdateManagement/RowCreate.cs (1 line):
	- line 322: IsNull = false,    // TODO: This doesn't properly consider null defaults


src/Microsoft.SqlTools.ManagedBatchParser/BatchParser/ExecutionEngineCode/BatchParserSqlCmd.cs (1 line):
	- line 74: // TODO: Halt the parser, should get/set variable have ParserAction.Abort/Continue (like original?)


src/Microsoft.SqlTools.ResourceProvider.Core/Firewall/FirewallErrorParser.cs (1 line):
	- line 87: //TODO: trace?


src/Microsoft.Kusto.ServiceLayer/ObjectExplorer/Nodes/NodeTypes.cs (1 line):
	- line 11: // TODO Consider replacing this with an auto-gen'd version


src/Microsoft.SqlTools.ServiceLayer/Management/Common/Utils.cs (1 line):
	- line 315: // TODO: Add constructor logic here


src/Microsoft.Kusto.ServiceLayer/Workspace/Workspace.cs (1 line):
	- line 274: // TODO: Assert instead?


src/Microsoft.SqlTools.ServiceLayer/Management/Common/STParameters.cs (1 line):
	- line 33: // TODO: Add constructor logic here


src/Microsoft.SqlTools.ServiceLayer/Connection/DatabaseLocksManager.cs (1 line):
	- line 56: * TODO: add the lock so not two process can get full access at the same time


src/Microsoft.Kusto.ServiceLayer/QueryExecution/Contracts/SaveResultsRequest.cs (1 line):
	- line 118: //TODO: define config for save as JSON


src/Microsoft.Kusto.ServiceLayer/Formatter/Contracts/DocumentFormatting.cs (1 line):
	- line 108: // TODO there may be other options passed by VSCode - format is


src/Microsoft.SqlTools.ServiceLayer/Agent/Jobs/JobData.cs (1 line):
	- line 1269: // TODO: this seems wrong. Why do it here and not in SMO?


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/Nodes/NodeObservableCollection.cs (1 line):
	- line 164: // TODO implement cleanup policy / pattern


src/Microsoft.Kusto.ServiceLayer/ServiceHost.cs (1 line):
	- line 168: // TODO: Figure out where this needs to go to be agnostic of the language


src/Microsoft.Kusto.ServiceLayer/ObjectExplorer/Nodes/NodeObservableCollection.cs (1 line):
	- line 164: // TODO implement cleanup policy / pattern


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/SmoModel/SmoParamterCustomNode.cs (1 line):
	- line 85: //TODO return parameters


src/Microsoft.SqlTools.Hosting/Hosting/Protocol/MessageWriter.cs (1 line):
	- line 50: // TODO: This method should be made protected or private


src/Microsoft.SqlTools.ServiceLayer/DisasterRecovery/RestoreOptionsHelper.cs (1 line):
	- line 302: //TODO: we could send back the error message so client knows the option is set incorrectly


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/SmoModel/SmoTableCustomNode.cs (1 line):
	- line 50: // TODO carbon issue 3125 enable "External" subtype once icon is ready. Otherwise will get missing icon here.


src/Microsoft.SqlTools.ServiceLayer/LanguageServices/ConnectedBindingQueue.cs (1 line):
	- line 152: //TODO: remove the binding context?


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/ObjectExplorerService.cs (1 line):
	- line 783: // TODO decide whether a cache is needed to handle lookups in elements with a large # children


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/SqlServerType.cs (1 line):
	- line 74: //TODO: not supporting SQL DW for on prem


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/SmoModel/SmoChildFactoryBase.cs (1 line):
	- line 210: // TODO Assess whether async operations node is required


src/Microsoft.SqlTools.ServiceLayer/Formatter/Impl/FormatterVisitor.cs (1 line):
	- line 126: // TODO:  need special logic here to deal with the placement of commas


src/Microsoft.SqlTools.ServiceLayer/ShowPlan/ShowPlanGraph/OperationTable.cs (1 line):
	- line 331: // TODO: may need to put a few more statements in here


src/Microsoft.SqlTools.ServiceLayer/Agent/Jobs/AgentProxyAccountActions.cs (1 line):
	- line 159: #if false  // @TODO - reenable subsystem code below


src/Microsoft.SqlTools.ManagedBatchParser/BatchParser/ExecutionEngineCode/ExecutionEngine.cs (1 line):
	- line 403: // TODO this is associated with Dacfx specific situations, so uncomment if need be


src/Microsoft.SqlTools.Hosting/Utility/Validate.cs (1 line):
	- line 44: // TODO: Debug assert here if lowerLimit >= upperLimit


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/SmoModel/SmoTreeNode.cs (1 line):
	- line 29: // TODO setup initialization


src/Microsoft.SqlTools.Hosting/Hosting/Protocol/ProtocolEndpoint.cs (1 line):
	- line 204: // TODO: Better default value here?


src/Microsoft.SqlTools.Hosting/Utility/UtilityServiceHost.cs (1 line):
	- line 145: // TODO: Figure out where this needs to go to be agnostic of the language


src/Microsoft.SqlTools.ServiceLayer/Formatter/Contracts/DocumentFormatting.cs (1 line):
	- line 108: // TODO there may be other options passed by VSCode - format is


src/Microsoft.SqlTools.ResourceProvider.DefaultImpl/AzureResourceManagementSession.cs (1 line):
	- line 63: //TODO: trace


src/Microsoft.SqlTools.ServiceLayer/Formatter/TSqlFormatterService.cs (1 line):
	- line 173: // TODO do not add if no formatting needed?


src/Microsoft.SqlTools.ServiceLayer/Admin/Database/DatabasePrototypeAzure.cs (1 line):
	- line 107: // TODO set from here should probably allow for the fact that System is a valid edition for


src/Microsoft.SqlTools.ManagedBatchParser/ReliableConnection/DbConnectionWrapper.cs (1 line):
	- line 95: TODO - IClonable does not exist in .NET Core.


src/Microsoft.SqlTools.ServiceLayer/ObjectExplorer/SmoModel/SqlHistoryTableQuerier.cs (1 line):
	- line 28: //TODO: have a better filtering here. HistoryTable is not available for SQL 2014.


src/Microsoft.SqlTools.ServiceLayer/EditData/UpdateManagement/CellUpdate.cs (1 line):
	- line 75: // @TODO: Microsoft.SqlServer.Types.SqlHierarchyId


src/Microsoft.SqlTools.ServiceLayer/Workspace/Workspace.cs (1 line):
	- line 274: // TODO: Assert instead?


src/Microsoft.SqlTools.ServiceLayer/ServiceHost.cs (1 line):
	- line 169: // TODO: Figure out where this needs to go to be agnostic of the language


src/Microsoft.SqlTools.Hosting/Hosting/Protocol/MessageReader.cs (1 line):
	- line 152: // TODO: Provide a more graceful shutdown path


src/Microsoft.SqlTools.ServiceLayer/QueryExecution/Contracts/SelectionData.cs (1 line):
	- line 12: /// TODO: Remove this in favor of buffer range end-to-end


src/Microsoft.SqlTools.ServiceLayer/Formatter/Impl/SqlCommonTableExpressionFormatter.cs (1 line):
	- line 54: // TODO: should we indent the AS statement and then decrement indent at the end?


src/Microsoft.SqlTools.ServiceLayer/Admin/Database/DatabasePrototype.cs (1 line):
	- line 309: // TODO: fix the exception in SMO


src/Microsoft.SqlTools.ResourceProvider.DefaultImpl/AzureResourceManager.cs (1 line):
	- line 148: // TODO ResourceGroup name