<?xml version="1.0"?>
<ownershipDocument>
    <schemaVersion>X0201</schemaVersion>

    <documentType>4</documentType><periodOfReport>2003-12-01</periodOfReport>
    <issuer>
        <issuerCik>0001126975</issuerCik>
        <issuerName>MAGELLAN MIDSTREAM PARTNERS LP</issuerName>
        <issuerTradingSymbol>MMP</issuerTradingSymbol></issuer><reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001246263</rptOwnerCik>
            <rptOwnerName>MAGELLAN MIDSTREAM HOLDINGS LP</rptOwnerName>
        </reportingOwnerId>
<reportingOwnerAddress><rptOwnerStreet1>One Williams Center</rptOwnerStreet1><rptOwnerStreet2/><rptOwnerCity>Tulsa</rptOwnerCity><rptOwnerState>OK</rptOwnerState><rptOwnerZipCode>74172</rptOwnerZipCode></reportingOwnerAddress><reportingOwnerRelationship><isDirector>1</isDirector><isOfficer>0</isOfficer><isTenPercentOwner>1</isTenPercentOwner><isOther>0</isOther></reportingOwnerRelationship></reportingOwner><reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001246261</rptOwnerCik>
            <rptOwnerName>MAGELLAN MIDSTREAM MANAGEMENT LLC</rptOwnerName>
        </reportingOwnerId>
<reportingOwnerAddress><rptOwnerStreet1/><rptOwnerStreet2/><rptOwnerCity/><rptOwnerState>OK</rptOwnerState><rptOwnerZipCode/></reportingOwnerAddress><reportingOwnerRelationship><isDirector>1</isDirector><isOfficer>0</isOfficer><isTenPercentOwner>1</isTenPercentOwner><isOther>0</isOther></reportingOwnerRelationship></reportingOwner><reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001122443</rptOwnerCik>
            <rptOwnerName>MADISON DEARBORN CAPITAL PARTNERS IV LP</rptOwnerName>
        </reportingOwnerId>
<reportingOwnerRelationship><isDirector>1</isDirector><isOfficer>0</isOfficer><isTenPercentOwner>1</isTenPercentOwner><isOther>0</isOther></reportingOwnerRelationship></reportingOwner><reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001181100</rptOwnerCik>
            <rptOwnerName>MADISON DEARBORN PARTNERS LLC</rptOwnerName>
        </reportingOwnerId>
<reportingOwnerRelationship><isDirector>1</isDirector><isOfficer>0</isOfficer><isTenPercentOwner>1</isTenPercentOwner><isOther>0</isOther></reportingOwnerRelationship></reportingOwner><reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001247497</rptOwnerCik>
            <rptOwnerName>RIVERSTONE HOLDINGS LLC</rptOwnerName>
        </reportingOwnerId>
<reportingOwnerRelationship><isDirector>1</isDirector><isOfficer>0</isOfficer><isTenPercentOwner>1</isTenPercentOwner><isOther>0</isOther></reportingOwnerRelationship></reportingOwner><reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001247527</rptOwnerCik>
            <rptOwnerName>CARLYLE RIVERSTONE ENERGY PARTNERS II LP</rptOwnerName>
        </reportingOwnerId>
<reportingOwnerRelationship><isDirector>1</isDirector><isOfficer>0</isOfficer><isTenPercentOwner>1</isTenPercentOwner><isOther>0</isOther></reportingOwnerRelationship></reportingOwner><reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001247529</rptOwnerCik>
            <rptOwnerName>C R ENERGY GP II LLC</rptOwnerName>
        </reportingOwnerId>
<reportingOwnerRelationship><isDirector>1</isDirector><isOfficer>0</isOfficer><isTenPercentOwner>1</isTenPercentOwner><isOther>0</isOther></reportingOwnerRelationship></reportingOwner><reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001247532</rptOwnerCik>
            <rptOwnerName>CARLYLE RIVERSTONE MLP HOLDINGS LP</rptOwnerName>
        </reportingOwnerId>
<reportingOwnerRelationship><isDirector>1</isDirector><isOfficer>0</isOfficer><isTenPercentOwner>1</isTenPercentOwner><isOther>0</isOther></reportingOwnerRelationship></reportingOwner><reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001249306</rptOwnerCik>
            <rptOwnerName>MADISON DEARBORN PARTNERS IV LP</rptOwnerName>
        </reportingOwnerId>
        <reportingOwnerAddress>
            <rptOwnerStreet1>THREE FIRST NATIONAL PLAZA STE 3800</rptOwnerStreet1>
            <rptOwnerStreet2></rptOwnerStreet2>
            <rptOwnerCity>CHICAGO</rptOwnerCity>
            <rptOwnerState>IL</rptOwnerState>
            <rptOwnerZipCode>60602</rptOwnerZipCode>
            <rptOwnerStateDescription></rptOwnerStateDescription>
        </reportingOwnerAddress>
            <reportingOwnerRelationship>
<isDirector>1</isDirector><isOfficer>0</isOfficer><isTenPercentOwner>1</isTenPercentOwner><isOther>0</isOther></reportingOwnerRelationship></reportingOwner><nonDerivativeTable><nonDerivativeTransaction><securityTitle><value>Common Units representing limited partner interests</value></securityTitle><transactionDate><value>2003-12-01</value></transactionDate><transactionCoding><transactionFormType>4</transactionFormType><transactionCode>M</transactionCode><equitySwapInvolved>0</equitySwapInvolved><footnoteId id="F1"/></transactionCoding><transactionAmounts><transactionShares><value>7830924</value></transactionShares><transactionPricePerShare><footnoteId id="F2"/></transactionPricePerShare><transactionAcquiredDisposedCode><value>A</value></transactionAcquiredDisposedCode></transactionAmounts><postTransactionAmounts><sharesOwnedFollowingTransaction><value>8910618</value></sharesOwnedFollowingTransaction></postTransactionAmounts><ownershipNature><directOrIndirectOwnership><value>D</value><footnoteId id="F3"/><footnoteId id="F4"/></directOrIndirectOwnership><natureOfOwnership><value/><footnoteId id="F3"/><footnoteId id="F4"/></natureOfOwnership></ownershipNature></nonDerivativeTransaction></nonDerivativeTable><derivativeTable><derivativeTransaction><securityTitle><value>Class B Common Units representing limited partner interests</value></securityTitle><conversionOrExercisePrice><footnoteId id="F5"/></conversionOrExercisePrice><transactionDate><value>2003-12-01</value></transactionDate><transactionCoding><transactionFormType>4</transactionFormType><transactionCode>M</transactionCode><equitySwapInvolved>0</equitySwapInvolved><footnoteId id="F1"/></transactionCoding><transactionAmounts><transactionShares><value>7830924</value></transactionShares><transactionPricePerShare><footnoteId id="F7"/></transactionPricePerShare><transactionAcquiredDisposedCode><value>D</value></transactionAcquiredDisposedCode></transactionAmounts><exerciseDate><footnoteId id="F8"/></exerciseDate><expirationDate><footnoteId id="F6"/></expirationDate><underlyingSecurity><underlyingSecurityTitle><value>Common Units representing limited partner interests</value></underlyingSecurityTitle><underlyingSecurityShares><value>7830924</value></underlyingSecurityShares></underlyingSecurity><postTransactionAmounts><sharesOwnedFollowingTransaction><value>0</value></sharesOwnedFollowingTransaction></postTransactionAmounts><ownershipNature><directOrIndirectOwnership><value>D</value><footnoteId id="F3"/></directOrIndirectOwnership><natureOfOwnership><value/><footnoteId id="F3"/></natureOfOwnership></ownershipNature></derivativeTransaction></derivativeTable><footnotes><footnote id="F1">The transaction described herein is also exempt under Rule 16b-6(b) as the conversion of a derivative security, thus Transaction Code C is also applicable.</footnote><footnote id="F2">Magellan Midstream Holdings, L.P. converted 7,830,924 shares of Class B Common Units into an equal number of Common Units on December 1, 2003.</footnote><footnote id="F3">These securities are owned by Magellan Midstream Holdings, L.P. (formerly WEG Acquisitions, L.P.), which is a member of a &quot;group&quot; for purposes of Section 13(d) of the Exchange Act including Magellan Midstream Management, LLC (formerly WEG Acquisition Mangement, LLC), Carlyle/Riverstone MLP Holdings, L.P., Carlyle/Riverstone Energy Partners II, L.P., C/R Energy GP II, LLC, Riverstone Holdings LLC, Madison Dearborn Capital Partners IV, L.P., Madison Dearborn Partners IV, L.P., Madison Dearborn Partners, L.L.C., and other entities not jointly filing on this Form 4.  Magellan Midstream Holdings, L.P. directly owns all securities reported on this Form 4, and all the other joint filers' ownership form is indirect, through one or more subsidiaries. </footnote><footnote id="F4">The joint filers are jointly filing this Form 4 and information regarding the joint filers other than Magellan Midstream Holdings, L.P. is set forth on Exhibit 99 to this Form 4.  The reporting persons disclaim beneficial ownership of the reported securities except to the extent of their pecuniary interests therein, and this report shall not be deemed an admission that the reporting persons are beneficial owners of the reported securities for purposes of Section 16 or for any other purpose.</footnote><footnote id="F5">1-for-1</footnote><footnote id="F6">None</footnote><footnote id="F7">Not Applicable</footnote><footnote id="F8">Immediately</footnote></footnotes><remarks/><ownerSignature><signatureName>John D. Chandler, Chief Financial Officer of Magellan Midstream Management, LLC, general partner of Magellan Midstream Holdings, L.P.</signatureName><signatureDate>2003-12-03</signatureDate></ownerSignature></ownershipDocument>

