Hi,
I've been using the 7.x-2.6rc1 branch to run a fairly large migration set from mssql. Every once and a while, a change on either our side or the source side leads to failures.

Right now we see something along the lines of "Migration failed with source plugin exception: mssql_query(): messag .. " in drush. It would be great if we could see the class name that failed it would really help us with debuging.

So

try {
      $this->source->rewind();
    }
    catch (Exception $e) {
      self::displayMessage(
        t('Migration failed with source plugin exception: !e',
          array('!e' => $e->getMessage())));
      return MigrationBase::RESULT_FAILED;
    }

Would be

try {
      $this->source->rewind();
    }
    catch (Exception $e) {
      self::displayMessage(
        t('Migration for !class failed with source plugin exception: !e',
          array('!e' => $e->getMessage(), '!class' => get_class($this))));
      return MigrationBase::RESULT_FAILED;
    }

I can try make a patch if anyone thinks this is a good idea.

Comments

mikeryan’s picture

Yes, please provide as a patch.

Thanks.

sergeypavlenko’s picture

Status: Active » Fixed

Hi

In the version 7.x-2.x this issue fixed:

Migration failed with source plugin exception: %e, in %file:%line

sergeypavlenko’s picture

Status: Fixed » Needs review
StatusFileSize
new1.34 KB

Sorry, is not fixed fully now.

Migration failed with source plugin exception: mssql_query(): message: The data types [error]
text and varchar are incompatible in the is operator. (severity 16)
File /dev/plugins/sources/mssql.inc, line
67
, in class="placeholder">/dev/modules/contrib/migrate/includes/base.inc: class="placeholder">707

Attached the patch to answer.

Status: Needs review » Needs work

The last submitted patch, 3: 2252839-3.patch, failed testing. View results

pifagor’s picture

Status: Needs work » Needs review
StatusFileSize
new1.16 KB

I recreate patch

pifagor’s picture

alex_optim’s picture

Status: Needs review » Reviewed & tested by the community

Patch good work for me.

pifagor’s picture

  • pifagor committed 778e676 on 7.x-2.x
    Issue #2252839 by pifagor, sergeypavlenko, alex_optim, mikeryan: Include...
pifagor’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.