Fix missing AND keyword in JDBCEBPFProfilingTaskDAO.getTaskRecord() SQL

[currenjin]

Fix missing and keyword in JDBCEBPFProfilingTaskDAO.getTaskRecord()

• Add a unit test to verify the fix.
• Update the CHANGES log.

JDBCEBPFProfilingTaskDAO.getTaskRecord() builds a SQL query with two WHERE conditions (table_column and logical_id), but the and keyword between them was missing.

The generated SQL looked like:

select * from ebpf_profiling_task where table_column = ?logical_id = ?

This is a SQL syntax error. Every call to getTaskRecord() fails at the database level.

Fix

Added the missing and between the two conditions:

// Before
" where " + JDBCTableInstaller.TABLE_COLUMN + " = ?" +
    EBPFProfilingTaskRecord.LOGICAL_ID + " = ?";

// After
" where " + JDBCTableInstaller.TABLE_COLUMN + " = ? and " +
    EBPFProfilingTaskRecord.LOGICAL_ID + " = ?";

• If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.

[wu-sheng]

@currenjin Thanks for finding these bugs. Curiously, are you going to deploy skywalking in some product env by using SQL DB? As I noticed, you are focusing on SQL-kind things.

[currenjin]

@wu-sheng Thanks for asking.
Yes, I’m considering SQL-based storage for real deployment scenarios, so I started from JDBC paths first.
I focused on small fixes that are easy to reproduce and verify with tests, and I’ll keep expanding to other areas as I get deeper into the codebase.