Uploaded image for project: 'MariaDB Connector/node.js'
  1. MariaDB Connector/node.js
  2. CONJS-131

Change CONJS-127 breaks our prodcution

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.4.0
    • Component/s: other
    • Labels:
      None
    • Environment:
      MariaDB Connector/node.jsCONJS-127

      Description

      This feature (node.js CONJS-127) in release in mariadb nodes connector 3.2.0/1 is really unfriendly for us, please help!

      We have been running a large production system for the last 4.5 years, prior to using mariadb node driver we used node mysql2, then we moved to mariadb node lib as soon as we could, this has been great for so many versions, but from 2.3.X it has broken our system.

      We have thousands of different queries, we can't change all our sql to avoid duplicate field names on a join (e..g. BadgeId from table 1 and table 2). The cost of testing all these changes is prohibitive (so many apps, so many reports, etc).

      This change (node.jsCONJS-127) prevents duplicate fields name in the results. MariaDb itself has no problem with this.

      We don't mind if the duplicate is removed from the results so long as an error is not thrown.

      Can this change not be put behind a configuration setting? Making it mandatory is really unfriendly and breaks compatibility.

      Please help! We have been super impressed with the node mariadb driver to date, but we cannot progress with it with this unfriendly breaking change, we would have to change drivers which is something we really don't want to do. We are a large production system and there may be others like us, Changing all our sql statements is almost impossible for us in practical terms, Please help us!!!

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              diego dupin Diego Dupin
              Reporter:
              Dallow Albert
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: