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

I'm pruning the excess data we're posting like version number #37

Merged
merged 9 commits into from
May 24, 2021

Conversation

brianoflondon
Copy link
Contributor

In line with discussion in #36 I don't see anyone using version number and reason so I don't see any point in putting them in for now.

Also fixed some typos and added de-duplication so the same URL won't appear in exactly the same custom_json ever (though the same one can appear in multiple subsequent podpings.

@brianoflondon
Copy link
Contributor Author

This needs more work.


# This is a global signal to shut down until RC's recover
# Stores the RC cost of each operation to calculate an average
HALT_THE_QUEUE = False
# HALT_TIME = [1,2,3]
HALT_TIME = [0,3,9,12,30,60,120]
HALT_TIME = [0,1,1,3,6,9,15]
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the reasoning for reducing HALT_TIME so dramatically?
If so, should additional HALT_TIME values be added?
HALT_TIME=[01,1,3,6,9,15,30,60,120]

# loop_test()
# for _ in range (5):
# old_data(0)
loop_test()
Copy link
Contributor

Choose a reason for hiding this comment

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

restore these lines to previous prior to commit,
the default shouldn't be to run the loop_test

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was kinda thinking I was the only one running this test script so I wasn't really paying too much attention to it for others.

This script is a little dangerous in that it has the potential to dump a ton of crap into Hive very quickly! I'm probably the only person with powerful enough Hive accounts right now to do this however :-)

@daveajones daveajones merged commit 870e4f8 into Podcastindex-org:main May 24, 2021
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.

None yet

3 participants