Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-4652

Some go documentation on the website needs to be updated

    XMLWordPrintableJSON

Details

    • Documentation
    • Status: Closed
    • Minor
    • Resolution: Done
    • 0.11.0
    • None
    • Go - Library, Website
    • None
    • macOS 10.14 

      Go 1.11.1 darwin/amd64

      Thrift 0.11.0

       

       

    Description

      The Go compiler generates invalid code for Thrift services, more specifically, calls to the `thrift.TProtocol`'s `Flush()` method are missing the `context.Context` argument:

      edam/NoteStore.go:18963:14: not enough arguments in call to oprot.Flush
      have ()
      want (context.Context)
      edam/NoteStore.go:18980:16: not enough arguments in call to oprot.Flush
      have ()
      want (context.Context)
      edam/NoteStore.go:18999:16: not enough arguments in call to oprot.Flush
      have ()
      want (context.Context)
      edam/NoteStore.go:19014:24: not enough arguments in call to oprot.Flush
      have ()
      want (context.Context)
      edam/NoteStore.go:19035:16: not enough arguments in call to oprot.Flush
      have ()
      want (context.Context)
      edam/NoteStore.go:19054:16: not enough arguments in call to oprot.Flush
      have ()
      want (context.Context)
      edam/NoteStore.go:19069:24: not enough arguments in call to oprot.Flush
      have ()
      want (context.Context)
      edam/NoteStore.go:19090:16: not enough arguments in call to oprot.Flush
      have ()
      want (context.Context)
      edam/NoteStore.go:19111:16: not enough arguments in call to oprot.Flush
      have ()
      want (context.Context)
      edam/NoteStore.go:19126:24: not enough arguments in call to oprot.Flush
      have ()
      want (context.Context)

       

      I used the following models:

       * https://github.com/evernote/evernote-thrift

       

       

      Attachments

        Issue Links

          Activity

            People

              jking3 James E. King III
              aztristian Tristian Azura
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: