>SELECT * > FROM Transactions >INNER JOIN (SELECT DataItems FROM udfParseCommaListToTable(@sTransTypeCodes, ',') AS TranCodes > ON Transactions.sTrans_Type_CD = TranCodes.DataItems >IMO, The 'IN' version is better because it clearly shows intent of the query. There's nothing to return from the second table which makes JOIN unnecessary. Not to mention that it should be 'Transactions.*'.
