Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

⛽ feat: add networking skills to roadmap #1

Merged
merged 1 commit into from
Apr 6, 2024

Conversation

alizmhdi
Copy link
Contributor

@alizmhdi alizmhdi commented Apr 5, 2024

add networking skills to roadmap

@mostafaghadimi
Copy link
Contributor

Hi @alizademhdi jan, thanks for your contribution. That’s LGTM. I wonder if it can be placed under the general software engineering skills or it should be considered as a separate topic. What’s your opinion?

@alizmhdi
Copy link
Contributor Author

alizmhdi commented Apr 5, 2024

Hey Mostafa.
In general, I think networking should be classified in a different class than general software engineering skills. Because the types of problems that are currently in the general skills section of software engineering are a little different from networking.

@mostafaghadimi
Copy link
Contributor

Approved. ✅

In case of agreement, please like this message.

Thanks
@ShAlireza @iamdorsa @niyushabaghayi @rezaaminimajd

@ShAlireza
Copy link
Contributor

ShAlireza commented Apr 6, 2024

I'm not sure if knowing about VPNs and firewalls is a crucial skill a data engineer should have.
BTW, I totally agree with a general knowledge of famous protocols like HTTP(S) and IP stack.

@mostafaghadimi
Copy link
Contributor

@ShAlireza one thing to mention is that many organizations use MFA (Multi-Factor Authentication) to keep their infrastructure secure. If we consider the data engineers as data infrastructure providers, I think it seems reasonable for them having idea about general concepts of security like firewall. In terms of AWS, we have security groups that Data/DevOps engineers should be familiar with.

@ShAlireza
Copy link
Contributor

Agree!

roadmap.yaml Outdated
Comment on lines 13 to 19
- Protocols: |
HTTP/HTTPS
TCP/IP
SSH
DNS
- VPN
- Firewalls
Copy link
Contributor

@mostafaghadimi mostafaghadimi Apr 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The value of networking should be key-value based (like dictionary) or list (valid yaml format).

@alizademhdi Please change this one if it is possible for you. One suggestion could be:

Networking:
  Protocols:
    - HTTP/HTTPS
    - TCP/IP
    - SSH
    - DNS
  Concepts:    
    - VPN
    - Firewalls

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed :)

@niyushabaghayi
Copy link
Contributor

I think adding VPN and Firewalls aren't necessary for being a Data Engineer. These names are so general and being deep to them is for other roles like SRE, Infraustructure Engineers, etc.
BTW, I agree to add a network topic with a protocols sub-title.

@alizademhdi

@alizmhdi alizmhdi force-pushed the FEAT/Networking-skills branch from b76074d to 88757df Compare April 6, 2024 11:11
Copy link
Contributor

@mostafaghadimi mostafaghadimi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done!

@mostafaghadimi mostafaghadimi changed the title feat: add networking skills to roadmap ⛽ feat: add networking skills to roadmap Apr 6, 2024
@mostafaghadimi mostafaghadimi merged commit c0a231d into data-burst:main Apr 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants