[KXI-65355] refactoring the execution methods

[Philip-Carneiro-KX]

Changes introduced by this PR

• Isolate the execution commands

[ecmel]

For Insights Connections I have the following observations:

PyKX works on output tab but not on kdb results nor Notebooks.

import random
import pykx as kx
 
x = [5 * i for i in range(45)] 
y = list(range(45))  
z = random.choices(['a', 'b', 'c'], k=45) 

kx.q('{([] x;y;z)}', x, y, z)

Notebook gives the following error message:

Execution stopped.
response.data.data.map is not a function

kdb results view does not output anything.

Errors not showing correctly on notebooks:

'error

with the following output:

Execution stopped.
Cannot read properties of undefined (reading 'values')

I will test kdb+ connections today but I was not able test DAPs since I do not have a server with.kxi.qsqlenabled right now.

@Philip-Carneiro-KX

[ecmel]

@Philip-Carneiro-KX Populate scratchpad on DAP with Python is still not working. Test Notebook

[ecmel]

• Populating scratchpad works on Notebooks but it notifies user with an information message, this information message should not be shown for notebooks.

• Python output on DAP or DAP process is not using a table, the output should be the same as scratchpad output for the same Python code:

[ecmel]

Test Results

Test for KXI-65355-2 branch against 1.14.0 main branch.

Execution

Type	kdb	IE Scratchpad	IE Q/SQL	IE Populate SP
q file	✓	✓	✓	x
py file	✓	✓	✓	x
sql file			x	x
q cell	✓	✓	✓	✓
py cell	✓	✓	✓	✓
sql cell			x	x
datasource			x	x

Legend
✓	Working
x	Not working

Other

• IE Populate SP shows superfluous information message for notebooks.
• Background starting of IE Scratchpad notification status message is not shown when needed.
• Notebook stop execution and connect offer can deadlock.

@Philip-Carneiro-KX

[kx-sonarqube]

Quality Gate passed

Issues
13 New issues
39 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
89.2% Coverage on New Code
0.8% Duplication on New Code

See analysis details on SonarQube

[ecmel]

Closing for now for the small value added vs. surfaced bugs.