VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • None
    • hdfs-client
    • None

    Description

      Firstly, we should be pulling the number of threads from options.io_threads (which should default to std::thread::hardware_concurrency()). The library should pass all tests always with io_threads set to 1 or to <a very high number>

      Secondly, we should have a constructor where the consumer doesn't need to manage the IOService explicitly, and the FileSystemImpl should create its own internally.

      Since the FileSystem is defined as being for a particular user/identity, there is a valid use case for the consumer to be constructing many FileSystem instances to represent many authenticated users in the same process, but want to share resources (notably have a single io_service shared amongst them all). In this case, the consumer would want to own the IOService and pass the same instance to multiple FileSystem instances.

      Attachments

        1. HDFS-10796.HDFS-8707.000.patch
          29 kB
          James Clampffer

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            James C James Clampffer
            bobhansen Bob Hansen
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment