Skip to content

Commit fb54f70

Browse files
author
David Hasani
committed
add telemetry
1 parent f424cf5 commit fb54f70

File tree

2 files changed

+24
-29
lines changed

2 files changed

+24
-29
lines changed

packages/core/src/amazonqGumby/chat/controller/controller.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -414,7 +414,7 @@ export class GumbyController {
414414
toJDKVersion
415415
) as CodeTransformJavaTargetVersionsAllowed,
416416
codeTransformProjectId: pathToProject === undefined ? telemetryUndefined : getStringHash(pathToProject),
417-
userChoice: 'Confirm',
417+
userChoice: 'Confirm-Language-Upgrade',
418418
})
419419

420420
const projectName = path.basename(pathToProject)
@@ -447,7 +447,7 @@ export class GumbyController {
447447

448448
telemetry.record({
449449
codeTransformProjectId: pathToProject === undefined ? telemetryUndefined : getStringHash(pathToProject),
450-
userChoice: 'Confirm',
450+
userChoice: 'Confirm-SQL-Conversion',
451451
})
452452

453453
const projectName = path.basename(pathToProject)

packages/core/src/codewhisperer/commands/startTransformByQ.ts

Lines changed: 22 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -280,40 +280,32 @@ export async function validateSQLMetadataFile(sctRulesData: any, message: any) {
280280
const sourceDB = sctRules['rules'][0]['locator']['sourceVendor'] as string
281281
const targetDB = sctRules['rules'][0]['locator']['targetVendor'] as string
282282
if (sourceDB.toUpperCase() !== DB.ORACLE) {
283-
transformByQState
284-
.getChatControllers()
285-
?.transformationFinished.fire({
286-
message: CodeWhispererConstants.invalidMetadataFileUnsupportedSourceVendor(sourceDB),
287-
tabID: message.tabID,
288-
})
283+
transformByQState.getChatControllers()?.transformationFinished.fire({
284+
message: CodeWhispererConstants.invalidMetadataFileUnsupportedSourceVendor(sourceDB),
285+
tabID: message.tabID,
286+
})
289287
return false
290288
} else if (targetDB.toUpperCase() !== DB.AURORA_POSTGRESQL && targetDB.toUpperCase() !== DB.RDS_POSTGRESQL) {
291-
transformByQState
292-
.getChatControllers()
293-
?.transformationFinished.fire({
294-
message: CodeWhispererConstants.invalidMetadataFileUnsupportedTargetVendor(targetDB),
295-
tabID: message.tabID,
296-
})
289+
transformByQState.getChatControllers()?.transformationFinished.fire({
290+
message: CodeWhispererConstants.invalidMetadataFileUnsupportedTargetVendor(targetDB),
291+
tabID: message.tabID,
292+
})
297293
return false
298294
} else if (targetDB.toUpperCase() !== transformByQState.getTargetDB()) {
299-
transformByQState
300-
.getChatControllers()
301-
?.transformationFinished.fire({
302-
message: CodeWhispererConstants.invalidMetadataFileTargetVendorMismatch(
303-
targetDB,
304-
transformByQState.getTargetDB()!
305-
),
306-
tabID: message.tabID,
307-
})
295+
transformByQState.getChatControllers()?.transformationFinished.fire({
296+
message: CodeWhispererConstants.invalidMetadataFileTargetVendorMismatch(
297+
targetDB,
298+
transformByQState.getTargetDB()!
299+
),
300+
tabID: message.tabID,
301+
})
308302
return false
309303
}
310304
} catch (e: any) {
311-
transformByQState
312-
.getChatControllers()
313-
?.transformationFinished.fire({
314-
message: CodeWhispererConstants.invalidMetadataFileUnknownIssueParsing,
315-
tabID: message.tabID,
316-
})
305+
transformByQState.getChatControllers()?.transformationFinished.fire({
306+
message: CodeWhispererConstants.invalidMetadataFileUnknownIssueParsing,
307+
tabID: message.tabID,
308+
})
317309
return false
318310
}
319311
return true
@@ -596,6 +588,9 @@ export async function startTransformationJob(uploadId: string, transformStartTim
596588
telemetry.record({
597589
codeTransformJobId: jobId,
598590
codeTransformRunTimeLatency: calculateTotalLatency(transformStartTime),
591+
codeTransformTarget: transformByQState.getMetadataPathSQL()
592+
? transformByQState.getTargetDB()
593+
: transformByQState.getTargetJDKVersion(),
599594
})
600595
})
601596
} catch (error) {

0 commit comments

Comments
 (0)