Fix #3989 notebook execution count should start empty (#4004)

- Fixed issues where we missed using actual execution count / starting from empty on this.

src/sql/parts/notebook/models/notebookModel.ts

@@ -315,7 +315,7 @@ export class NotebookModel extends Disposable implements INotebookModel {
 			cell_type: cellType,
 			source: '',
 			metadata: {},
-			execution_count: 1
+			execution_count: undefined
 		};
 		return this.notebookOptions.factory.createCell(singleCell, { notebook: this, isTrusted: true });
 	}

src/sql/workbench/services/notebook/common/sqlSessionManager.ts

@@ -357,7 +357,7 @@ export class SQLFuture extends Disposable implements FutureInternal {
 						content: <nb.IExecuteResult>{
 							output_type: 'execute_result',
 							metadata: {},
-							execution_count: 1,
+							execution_count: this._executionCount,
 							data: { 'application/vnd.dataresource+json': this.convertToDataResource(columns, d), 'text/html': this.convertToHtmlTable(columns, d) }
 						},
 						metadata: undefined,