Uploaded image for project: 'Kudu'
  1. Kudu
  2. KUDU-121

"Env" class is somewhat aimless

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Won't Fix
    • M4
    • n/a
    • util
    • None

    Description

      Per discussion in some recent code reviews (http://gerrit.ent.cloudera.com:8080/#/c/1531/ in particular) the Env class is sort of confusing what it is for – if it's for mocking only, then we should get rid of some of the pieces that are never mocked/substituted. If it's for platform portability then we probably should have a separate env_linux for linux-specific (non-posix) stuff. This JIRA is to clean up that mess.

      Attachments

        Activity

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

          People

            tlipcon Todd Lipcon
            tlipcon Todd Lipcon
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Issue deployment