GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Add port attribute to Connection object · Issue #96 · tableau/document-api-python

Add port attribute to Connection object #96

New issue
Copy link
New issue
Copy link
Closed
Closed
Add port attribute to Connection object#96
Copy link
Assignees
t8y8
Labels
enhancement
Milestone
2016.10 release (v0.5)

Description

@geordielad
geordielad
opened on Oct 16, 2016
Issue body actions

Can we add the port attribute to connection. This might be useful before the full document API is implemented. Some datasource changes could need to change the port in addition to the server etc. Some changes are a bit more involved but could include a port change. For example migrating from AWS RDS Postgres to Redshift might mean changing port from 5432 to 5439. A change of database variant could require other changes of course - schema in the relation for example.

Activity

  1. graysonarts
    added
    enhancement
    on Oct 16, 2016
  2. t8y8
    self-assigned this
    on Oct 16, 2016
  3. t8y8 commented on Oct 16, 2016

    @t8y8
    t8y8
    on Oct 16, 2016
    Contributor
    More actions

    I've got a patch for this but I need to confirm the semantics of port='' vs no port attribute and when they're valid or not.

  4. t8y8
    added a commit that references this issue on Oct 17, 2016

    Issue #96: Add port support (#97)

    1aa4fc0
  5. t8y8 commented on Oct 17, 2016

    @t8y8
    t8y8
    on Oct 17, 2016
    Contributor
    More actions

    @geordielad ask and ye shall receive. Enjoy!

    Closed in #97

  6. t8y8
    closed this as completedon Oct 17, 2016
  7. graysonarts
    added this to the 2016.10 release (v0.5) milestone on Oct 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

  • @t8y8
    t8y8

Labels

enhancement

Type

No type

Projects

No projects

    Milestone

    • 2016.10 release (v0.5)

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    You are visitor 01402503 — GeezerVison does not store the pages you look up.
    Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.