Uploaded image for project: 'MongoDB Database Tools'
  1. MongoDB Database Tools
  2. TOOLS-1411

seemingly rare data race in mongoimport

    • Type: Icon: Bug Bug
    • Resolution: Duplicate
    • Priority: Icon: Minor - P4 Minor - P4
    • None
    • Affects Version/s: 3.3.11
    • Component/s: mongoimport
    • Labels:
      None

      https://evergreen.mongodb.com/task_log_raw/mongo_tools_ubuntu_race_integration_f5d244c76e689a53046466644080bffa92759f35_16_08_22_21_33_06/0?type=T#L361

       [2016/08/22 17:51:23.429] ==================
       [2016/08/22 17:51:23.430] WARNING: DATA RACE
       [2016/08/22 17:51:23.430] Write by
       [2016/08/22 17:51:23.430] goroutine 33:
       [2016/08/22 17:51:23.430]   os.(*file).close()
       [2016/08/22 17:51:23.430]       /opt/go/src/os/file_unix.go:143 +0x1be
       [2016/08/22 17:51:23.430]   os.(*File).Close()
       [2016/08/22 17:51:23.430]       /opt/go/src/os/file_unix.go:132 +0x88
       [2016/08/22 17:51:23.430]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.(*MongoImport).ImportDocuments()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/mongoimport.go:329 +0x906
       [2016/08/22 17:51:23.430]   github.com/mongodb/mongo-tools/common/db.BuildWriteConcern()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/.gopath/src/github.com/mongodb/mongo-tools/common/db/write_concern.go:92 +0x6a
       [2016/08/22 17:51:23.430]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.(*MongoImport).configureSession()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/mongoimport.go:444 +0xe8
       [2016/08/22 17:51:23.430]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.(*MongoImport).importDocuments()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/mongoimport.go:362 +0x9b9
       [2016/08/22 17:51:23.430]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.(*MongoImport).ImportDocuments()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/mongoimport.go:329 +0x8ce
       [2016/08/22 17:51:23.430]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.TestImportDocuments.func1.16()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/mongoimport_test.go:752 +0x135
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.parseAction.func1()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/discovery.go:80 +0x29
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.(*context).conveyInner()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/context.go:261 +0x28c
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.(*context).Convey.func1()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/context.go:163 +0x7d
       [2016/08/22 17:51:23.430]   github.com/jtolds/gls._m()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/jtolds/gls/stack_tags.go:39 +0x39
       [2016/08/22 17:51:23.430]   github.com/jtolds/gls.mark1()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/jtolds/gls/stack_tags.go:18 +0x38
       [2016/08/22 17:51:23.430]   github.com/jtolds/gls._m()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/jtolds/gls/stack_tags.go:41 +0x90
       [2016/08/22 17:51:23.430]   github.com/jtolds/gls.markS()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/jtolds/gls/stack_tags.go:16 +0x38
       [2016/08/22 17:51:23.430]   github.com/jtolds/gls.addStackTag()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/jtolds/gls/stack_tags.go:13 +0x49
       [2016/08/22 17:51:23.430]   github.com/jtolds/gls.(*ContextManager).SetValues()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/jtolds/gls/context.go:92 +0x5d3
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.(*context).Convey()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/context.go:164 +0x4ce
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.Convey()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/doc.go:77 +0x74
       [2016/08/22 17:51:23.430]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.TestImportDocuments.func1()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/mongoimport_test.go:754 +0x1350
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.parseAction.func1()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/discovery.go:80 +0x29
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.(*context).conveyInner()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/context.go:261 +0x28c
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.rootConvey.func1()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/context.go:110 +0x1f0
       [2016/08/22 17:51:23.430]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.TestImportDocuments.func1()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/mongoimport_test.go:745 +0x1212
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.parseAction.func1()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/discovery.go:80 +0x29
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.(*context).conveyInner()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/context.go:261 +0x28c
       [2016/08/22 17:51:23.430]   github.com/smartystreets/goconvey/convey.rootConvey.func1()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/context.go:110 +0x1f0
       [2016/08/22 17:51:23.430]   github.com/jtolds/gls._m()
       [2016/08/22 17:51:23.430]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/jtolds/gls/stack_tags.go:39 +0x39
       [2016/08/22 17:51:23.430]   github.com/jtolds/gls.markS()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/jtolds/gls/stack_tags.go:16 +0x38
       [2016/08/22 17:51:23.431]   github.com/jtolds/gls.addStackTag()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/jtolds/gls/stack_tags.go:13 +0x49
       [2016/08/22 17:51:23.431]   github.com/jtolds/gls.(*ContextManager).SetValues()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/jtolds/gls/context.go:92 +0x5d3
       [2016/08/22 17:51:23.431]   github.com/smartystreets/goconvey/convey.rootConvey()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/context.go:113 +0x54b
       [2016/08/22 17:51:23.431]   github.com/smartystreets/goconvey/convey.Convey()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/vendor/src/github.com/smartystreets/goconvey/convey/doc.go:75 +0x52
       [2016/08/22 17:51:23.431]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.TestImportDocuments()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/mongoimport_test.go:775 +0x254
       [2016/08/22 17:51:23.431]   testing.tRunner()
       [2016/08/22 17:51:23.431]       /opt/go/src/testing/testing.go:473 +0xdc
       [2016/08/22 17:51:23.431] Previous read by goroutine 162:
       [2016/08/22 17:51:23.431]   os.(*File).read()
       [2016/08/22 17:51:23.431]       /opt/go/src/os/file_unix.go:228 +0x57
       [2016/08/22 17:51:23.431]   os.(*File).Read()
       [2016/08/22 17:51:23.431]       /opt/go/src/os/file.go:95 +0xbc
       [2016/08/22 17:51:23.431]   bufio.(*Reader).fill()
       [2016/08/22 17:51:23.431]       /opt/go/src/bufio/bufio.go:97 +0x364
       [2016/08/22 17:51:23.431]   bufio.(*Reader).Read()
       [2016/08/22 17:51:23.431]       /opt/go/src/bufio/bufio.go:207 +0x54d
       [2016/08/22 17:51:23.431]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.(*bomDiscardingReader).Read()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/common.go:123 +0x228
       [2016/08/22 17:51:23.431]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.(*sizeTrackingReader).Read()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/common.go:93 +0x91
       [2016/08/22 17:51:23.431]   github.com/mongodb/mongo-tools/common/json.(*Decoder).readValue()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/.gopath/src/github.com/mongodb/mongo-tools/common/json/stream.go:166 +0x90b
       [2016/08/22 17:51:23.431]   github.com/mongodb/mongo-tools/common/json.(*Decoder).ScanObject()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/.gopath/src/github.com/mongodb/mongo-tools/common/json/stream.go:67 +0xfa
       [2016/08/22 17:51:23.431]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.(*JSONInputReader).StreamDocument.func1()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/json.go:118 +0x3db
       [2016/08/22 17:51:23.431] Goroutine 33 (running) created at:
       [2016/08/22 17:51:23.431]   testing.RunTests()
       [2016/08/22 17:51:23.431]       /opt/go/src/testing/testing.go:582 +0xae2
       [2016/08/22 17:51:23.431]   testing.(*M).Run()
       [2016/08/22 17:51:23.431]       /opt/go/src/testing/testing.go:515 +0x11d
       [2016/08/22 17:51:23.431]   main.main()
       [2016/08/22 17:51:23.431]       _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/_test/_testmain.go:110 +0x210
       [2016/08/22 17:51:23.431] Goroutine 162 (running) created at:
       [2016/08/22 17:51:23.431]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.(*JSONInputReader).StreamDocument()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/json.go:135 +0xc4
       [2016/08/22 17:51:23.431]   _/data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport.(*MongoImport).importDocuments.func1()
       [2016/08/22 17:51:23.431]       /data/mci/7d5e2704245cbad4b93711c18eb731a1/src/mongoimport/mongoimport.go:386 +0x47
       [2016/08/22 17:51:23.431] ==================
      

            Assignee:
            Unassigned Unassigned
            Reporter:
            gabriel.russell@mongodb.com Gabriel Russell (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: