FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Jobs#list should return a list of Job objects · Issue #2 · splunk/splunk-sdk-python · GitHub

Repository navigation

Jobs#list should return a list of Job objects #2

Closed

Description

The Jobs#list method currently returns a list of SIDs, I think it should return a list of Job objects instead.

Currently I have to wrap the Jobs object and override the method to do this:

[Job(self._jobs.service, sid) for sid in self._jobs.list()]

Would there be any down side to created a Job object instead?

Activity

  1. itay commented on Oct 27, 2011

    Contributor

    Hey Nick - that's a great point, and something we're planning on changing.

    Overall, we're not super happy with the way we did the object model in Python, especially around things like this. In the Java and JavaScript SDKs (currently under development, get in touch if you want access) we've done it differently, like you suggested.

    I hope we'll have this fixed in the next couple of weeks.

    Itay

  2. ansman commented on Oct 27, 2011

    Author

    Cool.

    We're currently designing our framework to wrap everything SDK related so we can always change the behavior so it isn't too hard to fix :)

  3. reopened this on Oct 27, 2011
  4. itay commented on Oct 27, 2011

    Contributor

    Please do let us know if you have any requests for the SDK, or places where you think we can improve on.

  5. ansman commented on Oct 27, 2011

    Author

    While I'm working I'll make sure to note any oddities and stuff I want to change :)

  6. blovering commented on Mar 19, 2012

    Contributor

    This change has now been checked into the feature/collections branch – Collection.list (and the corresponding callable, Collection.call) now returns a list of member entities instead of a list of member entity names, for all collections.

    Once this has been code reviewed I will merge with develop.

  7. blovering commented on Apr 19, 2012

    Contributor

    This work has been merged into develop (some time ago).

  8. added a commit that references this issue on Oct 1, 2019
  9. added a commit that references this issue on Apr 15, 2026
    a29ff65
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions


      Back | FazBrowse Home | New Git URL