Skip to content

UmdTask461_DATA605_Spring2026_Real_Time_Stock_Market_Pipeline_Kafka_Spark#466

Closed
aashishvinod wants to merge 2 commits intogpsaggese:masterfrom
aashishvinod:UmdTask461_DATA605_Spring2026_Real_Time_Stock_Market_Pipeline_Kafka_Spark
Closed

UmdTask461_DATA605_Spring2026_Real_Time_Stock_Market_Pipeline_Kafka_Spark#466
aashishvinod wants to merge 2 commits intogpsaggese:masterfrom
aashishvinod:UmdTask461_DATA605_Spring2026_Real_Time_Stock_Market_Pipeline_Kafka_Spark

Conversation

@aashishvinod
Copy link
Copy Markdown

Closes #461

This PR adds the project folder for Real-Time Stock Market Data Pipeline
using Apache Kafka and Spark Structured Streaming.

Author: @aashishvinod
Reviewers: @gpsaggese @protocorn

@protocorn protocorn self-requested a review April 8, 2026 20:05
Copy link
Copy Markdown
Collaborator

@protocorn protocorn left a comment

Choose a reason for hiding this comment

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

This PR is not structured correctly.

It currently contains many unrelated changes outside your project folder, along with merge conflicts. Your PR should only include your own project files under:
class_project/data605/Spring2026/projects/<your_project_name>

Please update the PR so that it contains only your project submission and no unrelated repository changes.

@aashishvinod
Copy link
Copy Markdown
Author

aashishvinod commented Apr 8, 2026

Hi @protocorn and @gpsaggese,

I sincerely apologize for the confusion caused by the earlier PR. The original submission (PR #466) was made on time on Apr 1, 2026, but had structural issues due to Windows path conflicts that occurred during cloning, which caused unrelated files to appear in the PR.

I have now fixed this by creating a clean branch from upstream/master and opening a new PR #478 (#478) that contains only my project files under:
class_project/data605/Spring2026/projects/UmdTask461_DATA605_Spring2026_Real_Time_Stock_Market_Pipeline_Kafka_Spark/

I am closing this PR #466 and kindly request you to review PR #478 instead. I am sorry for any inconvenience this may have caused and I appreciate your patience and understanding.

Closes #461

Author: @aashishvinod
Reviewers: @gpsaggese @protocorn

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.

DATA605_Spring2026_Real_Time_Stock_Market_Pipeline_Kafka_Spark

2 participants