W-17037056 feat: add data update bulk/resume commands #1098
W-17037056 feat: add data update bulk/resume commands #1098mdonnalley merged 29 commits intomainfrom
data update bulk/resume commands #1098Conversation
50d03b6 to
172c7d5
Compare
[skip ci]
mdonnalley
left a comment
There was a problem hiding this comment.
just a few thoughts/suggestions - otherwise looks good 👍
src/bulkIngest.ts
Outdated
| const timeout = opts.async ? Duration.minutes(0) : opts.wait ?? Duration.minutes(0); | ||
| const async = timeout.milliseconds === 0; | ||
|
|
||
| const baseUrl = opts.conn.getAuthInfoFields().instanceUrl as string; |
There was a problem hiding this comment.
should we handle the possibility that .instanceUrl doesn't exist or undefined instead of using as string?
There was a problem hiding this comment.
I can't find the slack thread but IIRC we said stuff like instanceUrl should always be in auth files but changing the types ins sfdx-core now would be a big change, I'll wrap it in ensureString 👍🏼 .
| const result = execCmd<DataImportBulkResult>( | ||
| `data import bulk --file ${csvFile} --sobject Account --wait 10 --column-delimiter PIPE --json`, | ||
| { ensureExitCode: 0 } | ||
| ).jsonOutput?.result as DataImportBulkResult; |
There was a problem hiding this comment.
is as DataImportBulkResult just there to remove the | undefined? If so, why not use optional chaining instead of using an assertion?
There was a problem hiding this comment.
I think I started doing the assertion in other NUTs and just carried the assertion, I can remove it.
|
QA 🟢 🟡 It fails as expected but I'm wondering if there's a way to auto-detect the delimiter?? 🟢 🟡 As I was trying to figure out how to use the new command, I encountered this: The Looks like I needed to have an 🟢 🟢 handles aborted job
|
… into cd/bulk-update
data update bulk/resume commands data update bulk/resume commands
What does this PR do?
data update bulk/resumecommands--column-delimiterflag todata import bulkdata import bulklogic into a genericbulkIngestfunc that will be used by all bulkIngest-related commands (data import/update bulkin this PR,data delete/upsert bulkwill be migrated in a separate PR)What issues does this PR fix or reference?
@W-17037056@