Giter Club home page Giter Club logo

Comments (7)

ravisuhag avatar ravisuhag commented on July 23, 2024 2

and filename could be called path imo?

from meteor.

GrayFlash avatar GrayFlash commented on July 23, 2024 2

@mabdh makes sense, so should we have a config like this:

config:
    path: ./dir/some-dir/postgres_food_app_data.json
    format: json

Where format is for defining data format and we write the data in the expected format in the whatever file extension user provides??

from meteor.

mabdh avatar mabdh commented on July 23, 2024 1

What about we just use filename field and directory could be inferred from that field ?
e.g.

filename: ./dir/some-dir/postgres_food_app_data

from meteor.

GrayFlash avatar GrayFlash commented on July 23, 2024 1

What about we just use filename field and directory could be inferred from that field?

In this case we can also include the format then?? Cause just having the directory and filename patched together can be confusing if there is no extension at the end?

filename: ./dir/some-dir/postgres_food_app_data.json

from meteor.

GrayFlash avatar GrayFlash commented on July 23, 2024
sinks:
    name: file
    config:
        format: "yaml/json"
        filename: "postgres_food_app_data"
        output-dir: path/to/folder

I thought this should be a fine format for config with all the necessary info required for the file sink.
Also, supporting the json and yaml format is the expected behavior and not some other format, right?? @StewartJingga

Feel free to add any suggestions/feedback or additional expectations from this sink.
cc: @ravisuhag @mabdh @scortier

from meteor.

GrayFlash avatar GrayFlash commented on July 23, 2024

Also, have thought to keep the format and output-dir fields in config optional, and add some default values ourselves.
For eg:
format: json
output-dir = current

from meteor.

mabdh avatar mabdh commented on July 23, 2024

@GrayFlash but on the other hand, json and yaml are just a data format. they don't have to be coupled to the file extension. in this case, i don't think we should restrict file extension to the user but the best practice should be following what the data format is (we let user decide).

from meteor.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.